Skip to content

Conversation

JohnCremona
Copy link
Member

@JohnCremona JohnCremona commented Apr 24, 2023

📚 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

  • The title is concise, informative, and self-explanatory.
  • 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. (N/A)
  • I have updated the documentation accordingly. (N/A)

⌛ Dependencies

@JohnCremona
Copy link
Member Author

I have forgotten how to tell it where the new tarball eclib-20230424.tar.bz2 is on github.

@JohnCremona
Copy link
Member Author

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.

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 24, 2023

The lint errors are fixed in #35552

@JohnCremona
Copy link
Member Author

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?

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 24, 2023

Most likely unproblematic

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 24, 2023

@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:

Could not solve for environment specs
  The following packages are incompatible
  ├─ primecountpy   is installable with the potential options
  │  ├─ primecountpy 0.1.0 would require
  │  │  └─ python_abi 3.10.* *_cp310, which can be installed;
  │  ├─ primecountpy 0.1.0 would require
  │  │  └─ python_abi 3.7.* *_cp37m, which can be installed;
  │  ├─ primecountpy 0.1.0 would require
  │  │  └─ python_abi 3.8.* *_cp38, which can be installed;
  │  └─ primecountpy 0.1.0 would require
  │     └─ python_abi 3.9.* *_cp39, which can be installed;
  └─ python 3.11**  is uninstallable because it requires
     └─ python_abi 3.11.* *_cp311, which conflicts with any installable versions previously reported.

@isuruf
Copy link
Member

isuruf commented Apr 24, 2023

@tobiasdiez tobiasdiez added s: run conda ci Run the conda workflow on this PR. and removed s: run conda ci Run the conda workflow on this PR. labels Apr 26, 2023
@JohnCremona
Copy link
Member Author

So only the conda-related tests fail?

@JohnCremona
Copy link
Member Author

I just merged the latest develop into the original PR branch.

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 30, 2023

The immediate failures for anything labeled "conda ... environment-optional" are unrelated and can be ignored

@github-actions
Copy link

Documentation preview for this PR is ready! 🎉
Built with commit: 5f14b39

@mkoeppe
Copy link
Contributor

mkoeppe commented May 1, 2023

I've tested it using tox -e docker-fedora-38-standard-incremental -- eclib and docker-ubuntu-focal-standard-incremental -- eclib, builds fine.

@tobiasdiez tobiasdiez added this to the sage-10.0 milestone May 2, 2023
@tobiasdiez
Copy link
Contributor

tobiasdiez commented May 2, 2023

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 conda.txt file?

@JohnCremona
Copy link
Member Author

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 conda.txt file?

Maybe, I don't know. None of the other distros/*.txt mention a version.

@vbraun vbraun merged commit 2152954 into sagemath:develop May 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sage.libs.eclib.homspace fails to build on conda (macOS)
5 participants