cysignals: Add version 1.13.0 - #2467
Merged
Merged
Conversation
Signed-off-by: riseproject-dev[bot] <330740410+riseproject-dev[bot]@users.noreply.github.com>
Contributor
|
Upstream's pyproject.toml (new in 1.13.0) sets tool.cibuildwheel.enable = ["cpython-freethreading"], needed back when free-threaded wheels were opt-in. cibuildwheel v4.2.0 (pinned in this workflow) has removed that enable group entirely, so it hard-errors parsing an unknown group instead of ignoring it: cibuildwheel: Failed to parse enable group. Unknown enable group: cpython-freethreading. Free-threaded builds are unconditional now and cp314t is already in this matrix, so the setting is a no-op to override.
…f overriding via env The earlier CIBW_ENABLE="" env override didn't work: cibuildwheel merges the env var's enable groups into pyproject.toml's list rather than replacing it, so the unknown cpython-freethreading group (removed in cibuildwheel 3.0, which this repo pins) stayed in effect and the build kept failing with the same 'Failed to parse enable group' error. Patch the setting out of pyproject.toml instead, the same fix already used for pemja's identical issue (patches/pemja/0.6.2/0001-...). Added the missing python-wheels checkout + patch-apply steps this package's workflow didn't have yet.
The build now succeeds (both earlier fixes held) but the post-build check failed asserting a '.dist-info/LICENSE' entry. meson-python 0.22.0 (the version this workflow's unpinned build dependency resolves to today) implements PEP 639, which places declared license-files under <dist-info>/licenses/ rather than directly in <dist-info>/ - the convention older (setuptools-era) tooling used. The wheel does ship the licence, just at '.dist-info/licenses/LICENSE'; only the check's expected path was stale.
1.13.0 raised requires-python from >=3.12 (1.12.6) to >=3.13, so cibuildwheel selects zero build identifiers for cp312 and exits with 'No build identifiers selected' - a real, version-bump break, not something either of the previous two fixes (the enable-group patch, the licence-path check) touches. cp313/cp314/cp314t are unaffected.
luhenry
force-pushed
the
main
branch
3 times, most recently
from
October 1, 2026 15:22
39fb7ba to
a75cf68
Compare
luhenry
marked this pull request as ready for review
October 1, 2026 16:24
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Automatically generated by the nightly
check_versions.pyrun.cysignals v1.12.6 -> v1.13.0
Every
- version:entry added todocs/packages/cysignals.yamlis built by this PR's ownbuild-cysignals.ymlrun; merging publishes the wheels.