New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(test): make test.sh portable #155

Merged

assafmo merged 1 commit into scrtlabs:master from hackbg:fix/test-portable

Aug 17, 2023

Contributor

egasimus commented Aug 17, 2023

/bin/bash is not guaranteed to be there; the standard way to find Bash is by using /usr/bin/env and the PATH environment variable.

See: https://www.oreilly.com/library/view/bash-cookbook/0596526784/ch15s01.html
See: https://unix.stackexchange.com/questions/569964/posix-shell-scripts-shebang-bin-sh-vs-usr-bin-env-sh-any-difference#comment1060096_569964


          fix(test): make test.sh portable

e3ab584

'/bin/bash' is not guaranteed to be there;
the standard way to find Bash is by using
'/usr/bin/env' and the $PATH env variable.

See: https://www.oreilly.com/library/view/bash-cookbook/0596526784/ch15s01.html
See: https://unix.stackexchange.com/questions/569964/posix-shell-scripts-shebang-bin-sh-vs-usr-bin-env-sh-any-difference#comment1060096_569964

assafmo merged commit 03cd458 into scrtlabs:master

Member

assafmo commented Aug 17, 2023

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet