`cat <script.sh> | cargo run` should work as expected

nixpulvis / oursh

Your comrade through the perilous world of UNIX.

http://nixpulvis.com/oursh/oursh

MIT License

67 stars 6 forks source link

`cat <script.sh> | cargo run` should work as expected #3

Closed nixpulvis closed 5 years ago

nixpulvis commented 6 years ago

We don't currently handle reading from STDIN / Files / Pipes correctly. A decent abstraction here should make this look very clean.

nixpulvis commented 5 years ago

This doesn't seem to be an issue anymore with the new parsing. I even added some is_tty checks to remove the prompt from a piped command run like this. See ebe786ffd411e4247eab17b1c9974beb8b536972