path to perl hardcoded in gphelp (GP/PARI)

[mariah]

The first line of /usr/bin/gphelp has the path to perl hardcoded rather than using

/usr/bin/env perl.

This can be a problem at sites where perl is kept in a non-standard location.

[jdemeyer]

Actually, the path to perl is not hardcoded, the first line of gphelp is written when PARI is installed. So is there still a problem then?

[wjp]

That sounds like it could be a problem with binary releases of sage.

[jdemeyer]

Replying to wjp:

That sounds like it could be a problem with binary releases of sage.

I guess you're right. Anyway, it's a trivial fix to replace the first line of doc/gphelp.in by #!/usr/bin/env perl, but this should be included in #10430.

[was]

Jeroen points out that we should fix this exact issue in a whole bunch (=6) of files (and of course get this upstreamed eventually). All probably related to the documentation.

[jdemeyer]

Unfortunately some of the other files use

#!/usr/bin/perl -w

and there is no easy env replacement for that. Since -w only enables warnings, we could just remove that switch, but I don't think that's such a good idea. Suggestions?

[jdemeyer]

doc/gphelp.in:#!@perl@
misc/tex2mail.in:#!@perl@
src/desc/PARI/822.pm:#!/usr/bin/perl -w
src/desc/merge_822:#!/usr/bin/perl -w
src/desc/whatnow:#!/usr/bin/perl -w
src/desc/gen_proto:#!/usr/bin/perl -w
src/desc/doc_make:#!/usr/bin/perl
src/graph/plotport.c:#!/usr/bin/perl

[jdemeyer]

Just for the record: perl is not required to build PARI/GP (you get lots of warnings/errors during the build because the documentation can't be built but gp works fine).

[jdemeyer]

There is a patch in the updated PARI spkg at #10430, so I'm closing this as duplicate.