Ticket #9867 (closed defect: duplicate)
getting rid of endianness-dependent behaviour in GAP random sources
| Reported by: | dimpase | Owned by: | joyner |
|---|---|---|---|
| Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
| Component: | group theory | Keywords: | |
| Cc: | simonking | Work issues: | |
| Report Upstream: | Fixed upstream, in a later stable release. | Reviewers: | Dmitrii Pasechnik, Karl-Dieter Crisman |
| Authors: | Merged in: | ||
| Dependencies: | Stopgaps: |
Description (last modified by dimpase) (diff)
in the thread [How to deal with GAP's machine dependent random generator?] on sage-devel Simon King mentioned that GAP own random source dependes on endianness of the machine. While Sage sort of takes care of this in misc/randstate.pyx, it still does not fix GAP internals. So, to make it good and proper, we essentially add the fix in misc/randstate.pyx to GAPROOT/src/integer.c, and remove it from misc/randstate.pyx The updated gap spkg is here:
http://boxen.math.washington.edu/home/dima/packages/gap-4.4.12.p5.spkg
So one needs to install this spkg and apply the patch attached to the ticket. I don't seem to have access to a 64-bit big-endian system, so it would be great to test it there, just in case...
Attachments
-
trac_9876_gap_MT_endianness.patch
(1.6 KB) -
added by dimpase 3 years ago.
Change History
comment:2 Changed 3 years ago by dimpase
- Cc simonking added
- Report Upstream changed from Completely fixed; Fix reported upstream to Fixed upstream, but not in a stable release.
comment:4 Changed 8 months ago by dimpase
This will be fixed in #13211. Let us close this one as obsolete.
