Skip to content
This repository has been archived by the owner on Nov 30, 2024. It is now read-only.

Incomplete redirects aren't handled properly when followed by '|' #3

Closed
0x416c6941 opened this issue Nov 29, 2024 · 1 comment
Closed

Comments

@0x416c6941
Copy link

0x416c6941 commented Nov 29, 2024

This command:
cat >
Returns:

minishell: syntax error near unexpected token `newline'


Which is fine. However, e.g. this one:
cat > | cat
Is handled incorrectly, and instead just creates a file called "|", whilst the second command is ignored.

@mcombeau
Copy link
Owner

Hello,
Good catch ! Thank you for taking the time to find the issue and make a pull request, but we won't be merging it. We'd like this repository to stay as it was when we submitted the code for evaluation. Feel free to keep improving your own fork of it, though !
Good luck on your 42 projects ;)

@mcombeau mcombeau closed this as not planned Won't fix, can't repro, duplicate, stale Nov 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants