Opened 6 years ago

Closed 6 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, GitHub, GitLab) Commit: 2ecbab42c98a5b5830d851ea9eb93ccd20146ffd
Dependencies: Stopgaps:

Status badges

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 6 years ago.

Download all attachments as: .zip

Change History (9)

Changed 6 years ago by charpent

comment:1 Changed 6 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 6 years ago by fbissey

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

comment:4 Changed 6 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 6 years ago by jdemeyer

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

comment:6 Changed 6 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 6 years ago by jdemeyer

  • Status changed from needs_review to positive_review

comment:8 Changed 6 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.