isatty: command [ instead of command test

I don't know why but Go users keep having random tools installed into
PATH named `test`. Fixes #5665
This commit is contained in:
Aaron Gyes 2019-02-16 17:14:15 -08:00
parent 20c51b7da9
commit 05701a779b

View file

@ -27,5 +27,7 @@ function isatty -d "Tests if a file descriptor is a tty"
# Use `command test` because `builtin test` doesn't open the regular fd's. # Use `command test` because `builtin test` doesn't open the regular fd's.
# See https://github.com/fish-shell/fish-shell/issues/1228 # See https://github.com/fish-shell/fish-shell/issues/1228
command test -t "$fd" # Too often `command test` is some bogus Go binary, I don't know why. Use [ because
# it's less likely to be something surprising. See #5665
command [ -t "$fd" ]
end end