Repository navigation
maint: bump cibuildwheel, mpfr and flint - #283
oscarbenjamin merged 3 commits into
Conversation
Sorry, something went wrong.
|
The manylinux arm job is building too many wheels. Reported at pypa/cibuildwheel#2412 so I will leave this PR open as a demonstration and make a new separate PR to fix the Cython version. |
Sorry, something went wrong.
| # requires-python needs to keep in sync with this and also the list of Python | ||
| # versions the wheels are tested against in CI. | ||
| build = "cp311-* cp312-* cp313-* cp313t-*" # pp311-*" | ||
| build = "cp311-* cp312-* cp313-* cp313t-* cp314-* cp314t-* pp311-*" |
There was a problem hiding this comment.
By default, cibuildwheels will only build what's possible given requires-python.
I often find easier not to specify build in the configuration but only setting skip.
In this case, adding pp310-* to the skip config allows to drop the build config.
Sorry, something went wrong.
There was a problem hiding this comment.
Yes, that would be better. The comment above notes the awkwardness.
Sorry, something went wrong.
| skip = "*-win32 *-manylinux_i686 *-musllinux_*" | ||
|
|
||
| # Enable building for free-threaded CPython builds | ||
| enable = ["cpython-freethreading"] |
There was a problem hiding this comment.
The environment variable & toml setting are additive for enable, it might be better to keep the static config in pyproject.toml and only add cpython-prerelease in the workflow (you can also add a condition on tags if you don't want to forget to remove it before a release).
Sorry, something went wrong.
There was a problem hiding this comment.
Thanks. I don't think I want to have cpython-prerelease there at all actually (now that I have already confirmed that it works). For a moment it seemed like a good way to keep up to date with changes in 3.14 but actually I think it would be better just to have a job that builds fresh rather than messing with the wheel building configuration.
Sorry, something went wrong.
|
Okay, this has produced the expected number of wheels now: 5 wheels produced in 10 minutes: python_flint-0.7.1-cp311-cp311-manylinux_2_28_aarch64.whl 11,292 kB python_flint-0.7.1-cp312-cp312-manylinux_2_28_aarch64.whl 11,200 kB python_flint-0.7.1-cp313-cp313-manylinux_2_28_aarch64.whl 11,186 kB python_flint-0.7.1-cp313-cp313t-manylinux_2_28_aarch64.whl 11,279 kB python_flint-0.7.1-pp311-pypy311_pp73-manylinux_2_28_aarch64.whl 10,965 kB Thanks @mayeut for your help! |
Sorry, something went wrong.
Bump version constraints for Cython 3.1.