Opened 21 months ago

Closed 21 months ago

Last modified 21 months ago

#29026 closed defect (fixed)

update m4ri(e) to Jan 2020 versions

Reported by: dimpase Owned by:
Priority: major Milestone: sage-9.1
Component: packages: standard Keywords:
Cc: isuruf, malb, fbissey, jhpalmieri, mkoeppe, embray Merged in: Volker Braun
Authors: Dima Pasechnik Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: b2f41e8 (Commits, GitHub, GitLab) Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by dimpase)

tarballs: ​https://bitbucket.org/malb/m4rie/downloads/m4rie-20200115.tar.gz and ​​https://bitbucket.org/malb/m4ri/downloads/m4ri-20200115.tar.gz

Change History (19)

comment:1 Changed 21 months ago by dimpase

  • Cc isuruf malb fbissey jhpalmieri added

comment:2 Changed 21 months ago by dimpase

  • Description modified (diff)

comment:3 Changed 21 months ago by dimpase

  • Authors set to Dima Pasechnik
  • Branch set to u/dimpase/packages/m4ris2020
  • Commit set to c88e57a0af966dadc5810c6df29e0d7246d11129

New commits:

​fdd3894bump up versions of m4ri and m4rie
​c88e57aremove upstreamed m4ri patch

comment:4 Changed 21 months ago by dimpase

  • Cc mkoeppe embray added
  • Status changed from new to needs_review

comment:5 Changed 21 months ago by mkoeppe

Tested on macOS, test suite in src/matrix passes.

comment:6 Changed 21 months ago by mkoeppe

Fixes #29005.

comment:7 Changed 21 months ago by jhpalmieri

I'm also on OS X. Both packages pass their test suites, and make ptestlong succeeds. Doesn't fix #28944 or #28970, unfortunately, but that is not an obstacle to a positive review here.

comment:8 Changed 21 months ago by dimpase

anything more I can do here to facilitate reviewing?

comment:9 Changed 21 months ago by mkoeppe

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

comment:10 Changed 21 months ago by git

  • Commit changed from c88e57a0af966dadc5810c6df29e0d7246d11129 to b2f41e8f110497fc18b75deb1884c575b81b5db2
  • Status changed from positive_review to needs_review

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

​b2f41e8fix for m4ri/issues/72/a-possible-bug-in-mzd_submatrix

comment:11 Changed 21 months ago by dimpase

thsi should be fixing #28944 and #28970

comment:12 Changed 21 months ago by jhpalmieri

Again, on OS X they both pass their test suites and make ptestlong succeeds. Looks good to me. If someone can confirm on another platform or two, then positive review.

comment:13 Changed 21 months ago by dimpase

I'd bump up the package version to indicate it's got a patch

comment:14 Changed 21 months ago by mkoeppe

How should this be addressed in spkg-configure?

comment:15 Changed 21 months ago by dimpase

good question. e.g. we can tolerate buggy versions. or not.

comment:16 Changed 21 months ago by dimpase

I've asked upstream to make a new release. With it in place it will be easy to detect a good version.

comment:17 Changed 21 months ago by vbraun

  • Merged in set to Volker Braun
  • Status changed from needs_review to positive_review

Can go to a followup ticket imho...

comment:18 Changed 21 months ago by vbraun

  • Branch changed from u/dimpase/packages/m4ris2020 to b2f41e8f110497fc18b75deb1884c575b81b5db2
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:19 Changed 21 months ago by malb

  • Commit b2f41e8f110497fc18b75deb1884c575b81b5db2 deleted

FWIW I've made new releases:

Note: See TracTickets for help on using tickets.