Skip to content
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

renew_cert.sh: fix unusual shebang #87

Closed
wants to merge 1 commit into from
Closed

renew_cert.sh: fix unusual shebang #87

wants to merge 1 commit into from

Conversation

jwilk
Copy link
Contributor

@jwilk jwilk commented Feb 24, 2016

No description provided.

sh is normally lives in /bin, not in /usr/bin.
@StephenBrown2
Copy link

I'd say that's legit either way.

However to truly "fix" it, one might use the env binary, like so:

#!/usr/bin/env sh

@jwilk
Copy link
Contributor Author

jwilk commented Feb 25, 2016

Huh? I've never seen a shell script with a /usr/bin/sh or /usr/bin/env sh shebang before.
I've just checked all unixish systems I had access to (Debian 8.3, PLD Linux 3.0, FreeBSD 10.2, OpenBSD 5.7): all of them had /bin/sh; none of them had /usr/bin/sh.
Can you name one system released in the last decade that has /usr/bin/sh but no /bin/sh, or one that has /usr/bin/env but no /bin/sh?

@StephenBrown2
Copy link

Nope, just saying that's my convention for universality, since I'm used to python or perl or bash or zsh.

It's a good fix.

@diafygi
Copy link
Owner

diafygi commented Mar 26, 2016

Keeping it the way it is for now.

@diafygi diafygi closed this Mar 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants