-
-
Notifications
You must be signed in to change notification settings - Fork 435
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
Fedora 40 and gcc 14 #37698
Comments
I have that change on #37537 |
I have replicated this too |
I have tried in Fedora 40 with gcc package; |
The sagelib compile errors are cython/cython#2747 and possibly fixed in cython-3.10 |
Following @vbraun's comment, I have tried with cython-3.0.10. Some errors pointed out in #37537 have disappeared but the ones concerning plot didn't. I attach config.log and sagelib-10.4.beta5.log Probably the errors are there:
|
I've created #37939, with that I can at least build Sage on Fedora 40 |
…14.x <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Fedora 40 already ships GCC 14 Fixes sagemath#37698 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> - Depends on sagemath#37646 URL: sagemath#37537 Reported by: Matthias Köppe Reviewer(s):
If you put in the address line of the browser |
|
Still doesnt work for me, givaro and ppl fails. |
Which version of givaro? The package ppl is taken from the system in my case. Maybe |
i ran
output:
|
Whoops, wrong logfile for givaro. |
to be absolutely sure, i ran
and i can still reproduce the same problem |
Why do you have givaro 4.1.1? It should be 4.2.0. This causes the error for fflas_ffpack. I did not have the error with ppl since the system package is used. |
In my system, I have givaro 4.2.0 installed. I do not know why it is building 4.1.1 again, as i have applied the patch to upgrade it, as you mentioned |
Have you checked if the patch is really applied looking at
and then
Something similar for fflas-ffpack. The difference with your case is that I can build. I share the config.log in case it is useful: |
I believe i have the same result. Even though givaro-4.2.0 is installed from dnf, sage decides that SPKG should be installed |
I nuked the local cloned folder and got a fresh copy, it built; however it still gets givaro et al from source even though system packages are available |
I'm building with the Fedora libgmp for the record:
with that, I can build ppl just fine. |
…14.x <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Fedora 40 already ships GCC 14 Fixes sagemath#37698 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> - Depends on sagemath#37646 URL: sagemath#37537 Reported by: Matthias Köppe Reviewer(s):
…14.x <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Fedora 40 already ships GCC 14 Fixes sagemath#37698 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> - Depends on sagemath#37646 URL: sagemath#37537 Reported by: Matthias Köppe Reviewer(s):
…14.x <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Fedora 40 already ships GCC 14 Fixes sagemath#37698 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [ ] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> - Depends on sagemath#37646 URL: sagemath#37537 Reported by: Matthias Köppe Reviewer(s):
This is now in Sage 10.4.beta6, so testing should be easier |
I've created #38002 for the macOS/linbox/givaro issue |
I am having problems with system-wide flint (3.1.2),and gcc 14, as
or, similarly, replacing It's fixed in flint's main branch now (cf flintlib/flint#2027), though, but will linger for a while. |
I've fixed this one in #38227 |
I don't see how this can fix this - to fix this, you need to test that system flint can be used (but it cannot - at least not with numpy 2.0 - I didn't check other ones). So one needs a test in |
No. |
could you point out how your PR is fixing this ? |
You mean you didn't look? |
I didn't know where to look in the diff, as I apparently never knew about It's got to be in the PR description, too, by the way. |
Also, this has nothing to do with gcc 14. |
Like I said, this has absolutely nothing to do with gcc 14. |
(Do you read what's written?) |
Environment
Steps To Reproduce
Fedora 40beta has just arrived, I wanted to check if sage can be built. The first problem is that Fedora 40 is using gcc 14. Instead of using gcc package, I changed
spkg-configure.m4
inbuild/pkgs/gcc
andbuild/pkgs/gfortran
to allow gcc 14.The compilation gave an error for
givaro
; the patch in linbox-team/givaro#226 works for this. Next problem was with linbox. As a bypass, I used the changed files in #35148. The three packages have been successfully built.But I stuck at the compilation of
sagelib
, apparently withginac
but not sure, see the package logConfig log
config.log
Package logs
sagelib-10.4.beta0.log
Additional Information
No response
Checklist
Dependencies
The text was updated successfully, but these errors were encountered: