#30800 closed defect (fixed)

check system pari for qfisom bug

Reported by: dimpase Owned by:
Priority: blocker Milestone: sage-9.2
Component: build: configure Keywords:
Cc: mkoeppe, mjo, vbraun, isuruf Merged in:
Authors: Dima Pasechnik Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: f916905 (Commits, GitHub, GitLab) Commit: f916905c91ebb3d375e0b18966ae674078964b27
Dependencies: Stopgaps:

Status badges

Description (last modified by dimpase)

#27749 did not take care about testing system Pari for this bug. This ticket doess this.

Change History (11)

comment:1 Changed 22 months ago by dimpase

  • Cc mkoeppe mjo vbraun added
  • Description modified (diff)
  • Priority changed from major to blocker
  • Status changed from new to needs_review

comment:2 Changed 22 months ago by mjo

The test never should have been added in #27749 in the first place. It's an upstream bug. Now this test is copy/pasted in THREE places:

  • The upstream test suite
  • The sage test suite
  • spkg-configure.m4 for the affected package

Are we going to do this three times for every bug in every math library?

comment:3 Changed 22 months ago by dimpase

#27749 is done and dusted. The current ticket makes sure the system pari/gp is not buggy. We probably could have just as well checked the version of pari...

comment:4 Changed 22 months ago by dimpase

  • Cc isuru added

Isuru, by the way, this bug is present in the current Conda's pari 2.11.2.

comment:5 Changed 22 months ago by slelievre

Note that PARI 2.13.0 is out; upgrading to that new version is tracked at #30801.

comment:6 Changed 22 months ago by mkoeppe

  • Cc isuruf added; isuru removed

comment:7 Changed 22 months ago by mkoeppe

  • Branch changed from u/dimpase/packages/pari_check_qfisom to u/mkoeppe/packages/pari_check_qfisom

comment:8 Changed 22 months ago by mkoeppe

  • Commit changed from 8f1046073f3592898e496edb125fd4f29b4e9f55 to f916905c91ebb3d375e0b18966ae674078964b27

rebased


New commits:

f916905check system pari/gp for qfisom bug #2140

comment:9 Changed 22 months ago by mkoeppe

  • Reviewers set to Matthias Koeppe
  • Status changed from needs_review to positive_review

Looking good and seems to work (tested on ubuntu-bionic-standard and ubuntu-groovy-standard)

comment:10 Changed 22 months ago by isuruf

Thanks for the heads up. I update the pari package in conda to 2.11.4

comment:11 Changed 22 months ago by vbraun

  • Branch changed from u/mkoeppe/packages/pari_check_qfisom to f916905c91ebb3d375e0b18966ae674078964b27
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.