X Tutup
Skip to content

Move cibuildwheel configuration to pyproject.toml#30252

Open
QuLogic wants to merge 1 commit intomatplotlib:mainfrom
QuLogic:cibw-pyproject
Open

Move cibuildwheel configuration to pyproject.toml#30252
QuLogic wants to merge 1 commit intomatplotlib:mainfrom
QuLogic:cibw-pyproject

Conversation

@QuLogic
Copy link
Copy Markdown
Member

@QuLogic QuLogic commented Jul 3, 2025

PR summary

This allows developers to test out the build locally. With this, I was able to fix the test command by setting PIP_PREFER_BINARY to avoid re-building Pillow, which accidentally dropped manylinux2014 wheels in the latest release: python-pillow/Pillow#9057

Note also that we previously set CIBW_AFTER_BUILD, but this doesn't seem to be a valid setting. Thus I have dropped the twine check, which only tests the README rendering and so checking the sdist is sufficient. Additionally, I have commented out the license check, as we cannot do multiple licenses with meson-python without PEP639 (#28982).

PR checklist

@github-actions github-actions bot added the CI: Run cibuildwheel Run wheel building tests on a PR label Jul 3, 2025
@QuLogic QuLogic added the Build label Jul 3, 2025
@QuLogic
Copy link
Copy Markdown
Member Author

QuLogic commented Jul 3, 2025

Here for example, Pillow 11.2 (instead of the latest 11.3) was installed because of the missing manylinux2014 wheels: https://github.com/matplotlib/matplotlib/actions/runs/16042179733/job/45265805907?pr=30252#step:4:1161

This allows developers to test out the build locally. With this, I was
able to fix the test command by setting `PIP_PREFER_BINARY` to avoid
re-building Pillow, which accidentally dropped manylinux2014 wheels in
the latest release: python-pillow/Pillow#9057

Note also that we previously set `CIBW_AFTER_BUILD`, but this doesn't
seem to be a valid setting. Thus I have dropped the `twine check`, which
only tests the `README` rendering and so checking the sdist is
sufficient. Additionally, I have commented out the license check, as we
cannot do multiple licenses with meson-python without PEP639 (matplotlib#28982).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build CI: Run cibuildwheel Run wheel building tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

X Tutup