Skip to content
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

Updates to linbox-1.7.0, givaro-4.2.0, fflas-ffpack-2.5.0 #37938

Merged
merged 5 commits into from
May 12, 2024

Conversation

vbraun
Copy link
Member

@vbraun vbraun commented May 5, 2024

📚 Description

Draft PR taken from #35148. I rebased on top of 10.4 and added two more patches taken from upstream

@ClementPernet

closes #32959
closes #35148

@vbraun vbraun mentioned this pull request May 5, 2024
2 tasks
vbraun pushed a commit to vbraun/sage that referenced this pull request May 5, 2024
…-2.5.0

### 📚 Description

Draft PR taken from sagemath#35148. I rebased on top of 10.4 and added two more
patches taken from upstream

@ClementPernet

closes sagemath#32959
closes sagemath#35148

URL: sagemath#37938
Reported by: Volker Braun
Reviewer(s):
@vbraun vbraun requested review from mkoeppe and tornaria May 5, 2024 22:59
@vbraun
Copy link
Member Author

vbraun commented May 5, 2024

Does not build on 32-bit, but imho its much more urgent to fix Fedora 40 than to spend a lot of time on that. We can always do that later. Either way something is broken, but given the choice I'd rather have 32-bit broken.

@mkoeppe
Copy link
Member

mkoeppe commented May 5, 2024

Fine with me too to temporarily break *-i386-minimal

@mkoeppe
Copy link
Member

mkoeppe commented May 5, 2024

But i386 is just some one of several portability bugs that I reported to https://github.com/linbox-team/fflas-ffpack/issues and that are still open

@tornaria
Copy link
Contributor

tornaria commented May 5, 2024

FWIW, I'm packaging i686 without trouble, but we are on gcc 13.2.0. We regularly build and check sagemath on a few architectures including i686 (currently 10.4.beta5 at void-linux/void-packages#49571)

I've just rebuilt and checked givaro / fflas-ffpack / linbox from scratch (on i686) to make sure they are still working, and they seem fine.

BTW, when we switched to gcc 13 I had to add linbox-team/givaro#218, maybe you want to add that one too? (see: void-linux/void-packages#47712 (comment))

That is the only patch I have you don't (the other differences are: I don't have givaro 226 and I don't have linbox 310-backport, those seem to be needed for gcc 14).

@vbraun
Copy link
Member Author

vbraun commented May 6, 2024

So anyone wants to set it to positive review?

@mkoeppe
Copy link
Member

mkoeppe commented May 7, 2024

Reducing from blocker to critical because blockers still take effect as CI Fixes until this is merged:

@mkoeppe
Copy link
Member

mkoeppe commented May 8, 2024

@erentar
Copy link

erentar commented May 8, 2024

See #37698 (comment) , for some reason sage cannot detect installed givaro.
Might be relevant

@enriqueartal
Copy link
Contributor

See #37698 (comment) , for some reason sage cannot detect installed givaro. Might be relevant

It happens for both givaro and fflas-ffpack. In the cited config.log, system packages givaro and fflas-ffpack are marked as suitable (it is not the case for linbox, since flint is used as a package); but in the final summary they are marked as not suitable (there are the only switches).

vbraun pushed a commit to vbraun/sage that referenced this pull request May 9, 2024
…-2.5.0

### 📚 Description

Draft PR taken from sagemath#35148. I rebased on top of 10.4 and added two more
patches taken from upstream

@ClementPernet

closes sagemath#32959
closes sagemath#35148

URL: sagemath#37938
Reported by: Volker Braun
Reviewer(s):
@mkoeppe
Copy link
Member

mkoeppe commented May 9, 2024

It happens for both givaro and fflas-ffpack. In the cited config.log, system packages givaro and fflas-ffpack are marked as suitable (it is not the case for linbox, since flint is used as a package); but in the final summary they are marked as not suitable (there are the only switches).

See #37348 (comment)

@mkoeppe
Copy link
Member

mkoeppe commented May 9, 2024

Testing at https://github.com/mkoeppe/sage/actions/runs/8995091991

It builds OK on all Linux platforms except i386.
Let's merge it as suggested and sort out the breakage later.

@enriqueartal
Copy link
Contributor

For me, it works, the only issue is the strange situation with the system packages

vbraun pushed a commit to vbraun/sage that referenced this pull request May 11, 2024
…-2.5.0

### 📚 Description

Draft PR taken from sagemath#35148. I rebased on top of 10.4 and added two more
patches taken from upstream

@ClementPernet

closes sagemath#32959
closes sagemath#35148

URL: sagemath#37938
Reported by: Volker Braun
Reviewer(s):
vbraun pushed a commit to vbraun/sage that referenced this pull request May 11, 2024
…-2.5.0

    
### 📚 Description

Draft PR taken from sagemath#35148. I rebased on top of 10.4 and added two more
patches taken from upstream

@ClementPernet

closes sagemath#32959
closes sagemath#35148
    
URL: sagemath#37938
Reported by: Volker Braun
Reviewer(s):
vbraun pushed a commit to vbraun/sage that referenced this pull request May 12, 2024
…-2.5.0

    
### 📚 Description

Draft PR taken from sagemath#35148. I rebased on top of 10.4 and added two more
patches taken from upstream

@ClementPernet

closes sagemath#32959
closes sagemath#35148
    
URL: sagemath#37938
Reported by: Volker Braun
Reviewer(s):
@vbraun vbraun merged commit becc958 into sagemath:develop May 12, 2024
20 of 40 checks passed
@mkoeppe mkoeppe added this to the sage-10.4 milestone May 12, 2024
@kwankyu
Copy link
Collaborator

kwankyu commented May 13, 2024

I get

...
[fflas_ffpack-2.5.0] [spkg-install] libtool: link: g++ -std=gnu++11 -std=gnu++11 -dynamiclib  -o .libs/libfflas_c.1.dylib  .libs/fflas_lvl1.o .libs/fflas_lvl2.o .libs/fflas_lvl3.o .libs/fflas_sparse.o   -L/Users/kwankyu/GitHub/sage-dev/local/lib ./.libs/libfflas.dylib -lopenblas  -O2 -march=native -g -O2 -Wl,-rpath -Wl,/Users/kwankyu/GitHub/sage-dev/local/lib -Wl,-rpath -Wl,/Users/kwankyu/GitHub/sage-dev/local/lib   -install_name  /Users/kwankyu/GitHub/sage-dev/local/lib/libfflas_c.1.dylib -compatibility_version 2 -current_version 2.0 -Wl,-single_module
[fflas_ffpack-2.5.0] [spkg-install] Undefined symbols for architecture x86_64:
[fflas_ffpack-2.5.0] [spkg-install]   "Givaro::Integer::operator%(unsigned long long) const", referenced from:
[fflas_ffpack-2.5.0] [spkg-install]       Givaro::Modular<double, double, void>::init(double&, Givaro::Integer const&) const in fflas_lvl1.o
[fflas_ffpack-2.5.0] [spkg-install]       Givaro::Modular<double, double, void>::init(double&, Givaro::Integer const&) const in fflas_lvl2.o
[fflas_ffpack-2.5.0] [spkg-install]       Givaro::Modular<double, double, void>::init(double&, Givaro::Integer const&) const in fflas_lvl3.o
[fflas_ffpack-2.5.0] [spkg-install] ld: symbol(s) not found for architecture x86_64
[fflas_ffpack-2.5.0] [spkg-install] clang: error: linker command failed with exit code 1 (use -v to see invocation)
[fflas_ffpack-2.5.0] [spkg-install] make[7]: *** [libfflas_c.la] Error 1
[fflas_ffpack-2.5.0] [spkg-install] make[7]: *** Waiting for unfinished jobs....
[fflas_ffpack-2.5.0] [spkg-install] make[6]: *** [all-recursive] Error 1
[fflas_ffpack-2.5.0] [spkg-install] make[5]: *** [all-recursive] Error 1
[fflas_ffpack-2.5.0] [spkg-install] make[4]: *** [all-recursive] Error 1
[fflas_ffpack-2.5.0] [spkg-install] make[3]: *** [all] Error 2
[fflas_ffpack-2.5.0] [spkg-install] ********************************************************************************
[fflas_ffpack-2.5.0] [spkg-install] Error building fflas_ffpack-2.5.0

on mac with 3.5 GHz 6-Core Intel Xeon E5.

fflas_ffpack-2.5.0.log

Any help?

@vbraun
Copy link
Member Author

vbraun commented May 13, 2024

There is a -lgivaro missing, possibly intel mac only? New ticket, please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to givaro-4.2.0 fflas-ffpack-2.5.0 linbox-1.7.0
6 participants