-
-
Notifications
You must be signed in to change notification settings - Fork 656
update eclib to version 20230424 #35555
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
Conversation
I have forgotten how to tell it where the new tarball eclib-20230424.tar.bz2 is on github. |
Dammit, the source is at https://github.com/JohnCremona/eclib/releases/download/v20230424/eclib-20230424.tar.bz2 but the upstream tarball line in checksums.ini has https://github.com/JohnCremona/eclib/releases/download/VERSION/eclib-VERSION.tar.bz2 and the version has no v in front of it. Where is AI when you need it! I will edit checksums.ini accordingly. |
The lint errors are fixed in #35552 |
I thought they would have fixed elsewhere but saw no harm in fixing them too so all my tests would pass. Is that a problem? |
Most likely unproblematic |
@isuruf @tobiasdiez I am running our conda integration test on this PR (triggered by the label "s: run conda ci"), and I see a failure in https://github.com/sagemath/sage/actions/runs/4789352499/jobs/8517110991?pr=35555:
|
So only the conda-related tests fail? |
I just merged the latest develop into the original PR branch. |
The immediate failures for anything labeled "conda ... environment-optional" are unrelated and can be ignored |
Documentation preview for this PR is ready! 🎉 |
I've tested it using |
Setting as blocker since this is needed for the conda workflows (otherwise conda-installed eclib is rejected). Is is a good idea to set a fixed eclib version in the |
Maybe, I don't know. None of the other distros/*.txt mention a version. |
📚 Description
Should fix #35535 by updating eclib. The new version avoids using the template complex class with T=NTL's RR, which caused problems with newer compilers, and instead implements its own bigcomplex class.
📝 Checklist
⌛ Dependencies