Explicitly specify manylinux2014 in wheel building config #184
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ngoldbaum
commented
Jul 29, 2025
.github/workflows/ci.yaml
Outdated
| CIBW_MUSLLINUX_X86_64_IMAGE: ${{ matrix.musllinux_img || 'musllinux_1_2' }} | ||
| CIBW_MUSLLINUX_I686_IMAGE: ${{ matrix.musllinux_img || 'musllinux_1_2' }} | ||
| CIBW_MUSLLINUX_AARCH64_IMAGE: ${{ matrix.musllinux_img || 'musllinux_1_2' }} | ||
| CIBW_ENABLE: cpython-prerelease cpython-freethreading |
There was a problem hiding this comment.
neither of these are necessary for Python 3.14 or 3.14t as of the latest major version of cibuildwheel.
|
I don't see the artifacts uploaded. |
|
It looks like they're showing up attached to each job: https://github.com/python-cffi/cffi/actions/runs/16601223055#summary-46961152640 |
nitzmahone
approved these changes
Sep 5, 2025
nitzmahone
added a commit
that referenced
this pull request
Sep 5, 2025
* Explicitly specify --no-build-isolation as that's our expectation in these tests Previously we relied on pip to build the packages in non-PEP517 mode, which implied no build isolation. The latest `virtualenv` (with pypa/virtualenv#2868) won't include `wheel` in the virtualenv, which will mean that pip uses PEP-517 mode, which is isolated by default. (cherry picked from commit 24e42cb) * Add CFFI thread safety docs (#188) * Add CFFI thread safety docs * Delete incorrect statements * Add more links, examples, and suggestions about TSan * fix indentation in code example * Update doc/source/overview.rst Co-authored-by: Matti Picus <[email protected]> --------- Co-authored-by: Matti Picus <[email protected]> (cherry picked from commit e94a7b6) * Explicitly specify manylinux2014 in wheel building config (#184) (cherry picked from commit 078820c) * Enable more Windows pytest-run-parallel CI (#189) * enable windows pytest-run-parallel CI * pass skip-thread-unsafe * remove OS conditional (cherry picked from commit 3c61e14) * Misc CI env stabilization (#194) * Misc CI env stabilization * Specify explicit runner major image versions instead of `latest`. * Test only against versioned Python releases. Installing from arbitrary source commits with `-dev` is rarely worth the potential instability between runs. Specifying X.Y with `allow-prereleases: true` will use the latest packaged X.Y.Z release, falling back to the newest X.Y.0 pre-release if X.Y.0 has not yet been released. * correct manylinux image name typo (cherry picked from commit 51e276e) --------- Co-authored-by: Stefano Rivera <[email protected]> Co-authored-by: Nathan Goldbaum <[email protected]> Co-authored-by: Matti Picus <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While working on the wheel building CI for cryptography, I noticed that CFFI 2.0.0b1 is missing manylinux2014 wheels for some platforms. This is happening because cibuildwheel 3.0 changed the default manylinux image to manylinux_2_28. I think explicitly specifying manylinux2014 will create the manylinux2014 wheels that are currently missing.