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

Change shebang to use /usr/bin/env #307

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

Conversation

asymmetric
Copy link
Contributor

Makes the script more portable to systems where perl is not in /usr/bin, e.g. NixOS.

Makes the script more portable to systems where perl is not in /usr/bin, e.g. NixOS.
@digint
Copy link
Owner

digint commented Dec 26, 2020

If changing to /usr/bin/env perl, we would require to have a valid PATH environment, pointing to wherever perl is located.
This is not always the case, e.g. quite common within initrd.

Not sure if this would be the right thing to do anyways, as btrbk wants to be run by the "system perl", which is expected to be in /usr/bin (i know, some distros think differently, that's what makes it hard to find a decent compromise)

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.

2 participants