Opened 3 years ago

Closed 3 years ago

#21622 closed defect (fixed)

Pari no longer builds on systems where "." is not in the Perl path.

Reported by: charpent Owned by:
Priority: major Milestone: sage-7.4
Component: packages: standard Keywords: pari perl
Cc: Merged in:
Authors: François Bissey Reviewers: Jeroen Demeyer
Report Upstream: Fixed upstream, but not in a stable release. Work issues:
Branch: 2ecbab4 (Commits) Commit: 2ecbab42c98a5b5830d851ea9eb93ccd20146ffd
Dependencies: Stopgaps:

Description (last modified by jdemeyer)

Building pari depends on "." being in the Perl path, which is no longer the case in at least Debian. See attached log and ​this thread.

Release manager: merge together with #21582.

Attachments (1)

pari-2.8.0.alpha.p0.log​ (8.8 KB) - added by charpent 3 years ago.

Download all attachments as: .zip

Change History (9)

Changed 3 years ago by charpent

comment:1 Changed 3 years ago by fbissey

Parking this here until I (or someone else) can deal with it. The behavior was introduced by debian changes to perl in advance. Therefore debian has to patch its own pari package. ​https://sources.debian.net/patches/pari/2.7.6-2/perl-no-dot-in-inc/

comment:2 Changed 3 years ago by fbissey

Looks like an upstream patch too, so we probably can cherrypick it from pari's git tree.

comment:4 Changed 3 years ago by fbissey

  • Authors set to François Bissey
  • Branch set to u/fbissey/pari_perl
  • Commit set to 2ecbab42c98a5b5830d851ea9eb93ccd20146ffd
  • Status changed from new to needs_review

Just adding upstream patch and bumping the version.

New commits:

​2ecbab4upstream for perl @INC

comment:5 Changed 3 years ago by jdemeyer

  • Report Upstream changed from N/A to Fixed upstream, but not in a stable release.

comment:6 Changed 3 years ago by jdemeyer

  • Description modified (diff)
  • Reviewers set to Jeroen Demeyer

Note that #21582 also bumps the PARI version. There should be no conflict since both tickets make the same change to package-version.txt.

comment:7 Changed 3 years ago by jdemeyer

  • Status changed from needs_review to positive_review

comment:8 Changed 3 years ago by vbraun

  • Branch changed from u/fbissey/pari_perl to 2ecbab42c98a5b5830d851ea9eb93ccd20146ffd
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.