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
Upgrade lrslib to fix total brokenness on 32-bit systems #21451
Comments
This comment has been minimized.
This comment has been minimized.
comment:3
I can reproduce the problem too (while running the tests with almost all optional spkgs installed for SageDebianLive) ! |
comment:4
I can confirm that it's broken. I'm working on a fix. |
Author: Matthias Koeppe |
This comment has been minimized.
This comment has been minimized.
Commit: |
New commits:
|
Reviewer: Jeroen Demeyer |
Replying to @jdemeyer:
I doubt that this step will be acceptable. Better make the tarball with correct name in the first place, or change the version number in Sage to |
comment:10
It seems that this indeed works on 32-bit systems. I am going to set this to positive_review but you are still encouraged to do something about the naming of the tarball. |
comment:11
Note that i am also currently testing it (will take a few hours). |
Changed reviewer from Jeroen Demeyer to Jeroen Demeyer, Thierry Monteil |
comment:12
Works also for me (spkg-check + test long), both on 32 and 64 bits ! |
Changed branch from u/mkoeppe/lrslib_totally_broken_on_32_bit_systems to |
Changed keywords from none to sdl |
While compiling, there are some very suspicious warnings:
(several more warnings of the same kind)
Fixed in new lrslib version:
https://github.com/mkoeppe/lrslib/releases/download/lrslib-062%2Bautotools-2016-09-08/lrslib-062.autotools-2016-09-08.tar.gz (download into upstream and rename to lrslib-062+autotools-2016-09-08.tar.gz)
CC: @mkoeppe
Component: packages: optional
Keywords: sdl
Author: Matthias Koeppe
Branch/Commit:
963c748
Reviewer: Jeroen Demeyer, Thierry Monteil
Issue created by migration from https://trac.sagemath.org/ticket/21451
The text was updated successfully, but these errors were encountered: