Default the perl interpreter to the first one in $PATH... #28
Conversation
…local/sbin:/usr/bin:/usr/sbin:/bin:/sbin:/home/rassilon/.composer/vendor/bin:/home/rassilon/.local/bin:/home/rassilon/bin using the 'env' command. Previously, each file specified something like /usr/bin/perl, /usr/local/bin/perl, /u1/local/bin/perl, /tools/bin/perl, etc.
|
@rmhubley Sorry, I forgot to tag you for review. |
|
I have avoided doing this because of dependency checking. The configure script takes care of performing perl module dependency checking prior to setting the interpreter at the top of each script. So, in a sense the first line of these scripts could be anything to begin with. |
|
Sure, you expect the configuration script to be run, but that doesn't happen when this is installed via Spack (https://spack.io/) for a HPC environment. In that case, using "/usr/bin/env perl" is basically required. Also, if the configuration script rewrites the first line during normal usage, why is changing the interpreter to be consistent a problem? |
| @@ -1,4 +1,4 @@ | |||
| #!/usr/local/bin/perl -w | |||
| #!/usr/bin/env perl -w | |||
There was a problem hiding this comment.
This line and all others in the modules (*.pm) should be removed.
...using the 'env' command. Previously, each file specified something like /usr/bin/perl, /usr/local/bin/perl, /u1/local/bin/perl, /tools/bin/perl, etc.