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

remove Cython extension from future Python-bundle-PyPI easyconfigs #20524

Open
smoors opened this issue May 14, 2024 · 1 comment
Open

remove Cython extension from future Python-bundle-PyPI easyconfigs #20524

smoors opened this issue May 14, 2024 · 1 comment
Labels

Comments

@smoors
Copy link
Contributor

smoors commented May 14, 2024

rationale:

  • Cython should be a builddep not a dep in most cases
  • some Python packages require a specific Cython version
  • the installation Python-bundle-PyPI itself currently does not need Cython

it's probably too risky to do this for the existing Python-bundle-PyPI easyconfigs, but i want to make sure the next version does not have Cython in it (or only as a builddep if needed).

@smoors smoors added the change label May 14, 2024
@WilleBell
Copy link
Contributor

This would be useful for #20525 where Cython had to be included as a dependency after Python-bundle-PyPI because sci-kit-learn v1.4.2 requires a newer Cython version to build. Same workaround was used for Cartopy-0.22.0-foss-2023a.eb . They workaround works, but having 2 versions in the path even though only 1 is used doesn't seem ideal. Neither is having a dependency that is only needed as build dependency.

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

No branches or pull requests

2 participants