-
-
Notifications
You must be signed in to change notification settings - Fork 490
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gap 4.6.3 upgrade #14039
Comments
This comment has been minimized.
This comment has been minimized.
comment:2
Would this require also an upgrade of |
Author: Jeroen Demeyer |
This comment has been minimized.
This comment has been minimized.
comment:5
Volker, any idea what to do about this:
Running |
comment:6
LibGAP needs to be kept in sync since it uses the GAP library from the stand-alone GAP install. I can update libgap... |
comment:7
Replying to @vbraun:
OK, sounds good. |
comment:8
|
comment:9
I've added the corresponding libgap spkg. I didn't make any real changes, only applied the GAP 4.5.7->4.6.2 diffs. |
Changed author from Jeroen Demeyer to Jeroen Demeyer, Volker Braun |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
comment:11
For the record: I based the libgap spkg on the libgap version that awaits review at #14014. There are a few doctests that fail because of changed GAP notation for some group elements. |
Dependencies: #14014 |
This comment has been minimized.
This comment has been minimized.
comment:14
Does this really depend on #14014? If not, could you upgrade the libGAP spkg to 4.6.3? |
This comment has been minimized.
This comment has been minimized.
comment:16
gap, libgap, database_gap, and gap_packages must be kept in sync. |
This comment has been minimized.
This comment has been minimized.
comment:18
Replying to @vbraun:
Sure. I can do all packages except for libGAP. |
comment:19
The patch depends on #14014 |
This comment has been minimized.
This comment has been minimized.
comment:21
The siginterrupt.patch has been upstreamed |
This comment has been minimized.
This comment has been minimized.
Attachment: trac_14039_doctest_fixes.patch.gz Updated patch |
comment:22
All doctests pass for me... |
comment:23
I combined those packages and patches with those of ticket #14014, and things compile and doctest (except the quadratic forms ones which fail on this box anyway). I'm not sure I can give a positive review though: the patches change the result of many tests, and I have no idea if they really replace a correct value by another correct value... |
comment:24
Superseded by #14682 |
Changed author from Jeroen Demeyer, Volker Braun to none |
Reviewer: Travis Scrimshaw |
And now MPIR defines
__GNU_MP_RELEASE
so the__GMP_MP_RELEASE
hack is not needed anymore here or in gap spkg.Depends on #14014
Component: packages: standard
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/14039
The text was updated successfully, but these errors were encountered: