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

Use universal she-bang for Debian-based systems #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Tvrqvoise
Copy link

@Tvrqvoise Tvrqvoise commented May 29, 2018

On some versions of Debian (and thus, Ubuntu and friends), node is installed as nodejs. This means that the default node shebang does not work, since there is no node binary.

The fix here is to use the bourne shell shebang and then invoke the remainder of the script as node or nodejs, depending on what's installed on the system. This is based off of this Unix StackExchange answer.

Because this happens in the shebang, it is outside of the try / catch, and therefore fails any git operations, regardless of if there are hooks installed or not.

On some versions of Debian (and thus, Ubuntu and friends), `node` is installed as `nodejs`.  This means that the default node shebang does not work, since there is no `node` binary.  This shebang runs as shell, and then invokes the remainder of the script as node or nodejs, depending on what's installed on the system.

Because this happens in the shebang, it is outside of the `try / catch`, and therefore fails any git operations, regardless of if there are hooks installed or not.
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

1 participant