fix(utimes): change /dev/stdin to -#1124
Conversation
As not all environments have /dev/stdin. See ish-app/ish#944
Thanks for your contribution. |
There was a problem hiding this comment.
Thanks for your contribution - I'll approve these changes, but I'm not certain they are necessary? According to the manpage:
Bash handles several filenames specially when they are used in redirec‐tions, as described in the following table. If the operating system on which bash is running provides these special files, bash will use them; otherwise it will emulate them internally with the behavior described below.
/dev/stdin: File descriptor 0 is duplicated
Since our shebang points to Bash, I think this shouldn't be a problem. :)
@hyperupcall It’s not a bash thing, the device doesn’t exist in the OS: Note that since iSH (and CloudLinux) don’t support See ish-app/ish#164 |
|
@rasa I don't quite understand - your example code is not running under Bash. I don't expect |
It is, as |
|
I see - perhaps I misread the Bash man page as those /dev/stdin files are not being used for redirection. The Thanks for the fix |
As not all environments have
/dev/stdin. See ish-app/ish#944Sorry ‘bout that.