Repository navigation
ci(release): build manylinux wheels for the matrix Python version - #611
Merged
Merged
Conversation
#609 moved the Linux gnu Python wheels into the manylinux_2_28 container but kept `-i python`. Inside the container a bare `python` is the image's default interpreter, not the runner's setup-python one, so every Linux job built a cp39 wheel; the 3.12 job then could not install it and the v5.2.2 Release run failed (and fail-fast cancelled the rest; nothing was published, since every publish job waits on the full build). Name the interpreter explicitly on Linux (`-i python<version>`), as the musl job already does; Windows/macOS still build on the runner with `-i python`.
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.
Problem
The v5.2.2 Release run (38894106821) failed in
build-python-sdk (3.12, ubuntu-24.04-arm):#609 moved the Linux gnu wheels into the
manylinux_2_28container but kept-i python. Inside the container, a barepythonis the image's default interpreter, not the runner's setup-python one. So every Linux job built a cp39 wheel no matter what its matrix version was, and the 3.12 job couldn't install it.fail-fastthen cancelled the rest of the matrix.Nothing was published. Every
publish-*job waits on the aggregatebuildjob, which needs all builds to succeed.The Node.js half of #609 (
--use-napi-cross) built fine: both Linux gnubuild-nodejs-sdkjobs succeeded in that run.Fix
On Linux, name the interpreter explicitly:
-i python${{ matrix.python-version }}. This is what thebuild-python-sdk-musljob already does inside itsmusllinux_1_2container. Windows and macOS build on the runner itself and keep-i python.Verification
release.ymlparses as valid YAML.-i python<version>form already works for the musl wheels in every release run.release.CI-only change; no CHANGELOG entry.
🤖 Generated with Claude Code