#10559 closed defect (duplicate)
path to perl hardcoded in gphelp (GP/PARI)
Reported by: | mariah | Owned by: | tdb |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | relocation | Keywords: | pari gphelp |
Cc: | Merged in: | ||
Authors: | Reviewers: | Jeroen Demeyer | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
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.
Change History (9)
comment:1 Changed 11 years ago by
- Keywords pari gphelp added
- Status changed from new to needs_info
comment:2 follow-up: ↓ 3 Changed 11 years ago by
That sounds like it could be a problem with binary releases of sage.
comment:3 in reply to: ↑ 2 Changed 11 years ago by
- Status changed from needs_info to needs_work
comment:4 Changed 11 years ago by
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.
comment:5 Changed 11 years ago by
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?
comment:6 Changed 11 years ago by
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
comment:7 Changed 11 years ago by
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).
comment:8 Changed 11 years ago by
- Resolution set to duplicate
- Reviewers set to Jeroen Demeyer
- Status changed from needs_work to closed
There is a patch in the updated PARI spkg at #10430, so I'm closing this as duplicate.
comment:9 Changed 11 years ago by
- Milestone set to sage-duplicate/invalid/wontfix
Actually, the path to
perl
is not hardcoded, the first line ofgphelp
is written when PARI is installed. So is there still a problem then?