-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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.
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
bootstrap.sh on FreeBSD #6
Comments
There's indeed no need for bash in this case. Everything in that file is just plain Bourne shell syntax, switching to #!/bin/sh would make it easier to get it up and running on many commercial Unixes and BSDs. |
This should be changed to sh with the pull request Evan merged in issue #8, so this can be closed as well. |
Thanks ashemedai, this is now fixed. |
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I thing that in bootstrap.sh #!/bin/bash should be replaced by #!/bin/sh, because on FreeBSD and some other system /bin/bash is not available by default. And this script don't use any bashism that don't work on standard /bin/sh.
Of course I can install bash from FreeBSD ports, but this goes to /usr/local/bin/bash,
so I still need change this line. Another option is using #!/usr/bin/env bash for shearching in $PATH dirs, but #!/bin/sh still looks better for me.
The text was updated successfully, but these errors were encountered: