RepeatMasker icon indicating copy to clipboard operation
RepeatMasker copied to clipboard

Default the perl interpreter to the first one in $PATH...

Open isu-rassilon opened this issue 6 years ago • 3 comments

...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.

isu-rassilon avatar Jun 07 '19 17:06 isu-rassilon

@rmhubley Sorry, I forgot to tag you for review.

isu-rassilon avatar Jun 07 '19 17:06 isu-rassilon

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.

rmhubley avatar Oct 30 '19 18:10 rmhubley

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?

isu-rassilon avatar Nov 02 '19 14:11 isu-rassilon