Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add a Python matrix to ensure the bindings build on all supported versions #1871

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

nicholasjng
Copy link
Contributor

Also contains a run of pre-commit autoupdate, and a bump of cibuildwheel to its latest tag for CPython 3.13 support.

But, since we build for 3.10+ with SABI from 3.12 onwards, we don't even need a dedicated Python 3.13 build job - the wheels from 3.12 can be reused.

@dmah42
Copy link
Member

dmah42 commented Oct 28, 2024

3.13 is unhappy.

@nicholasjng
Copy link
Contributor Author

Ah yeah, I didn't commit the 3.13 toolchain def in MODULE.bazel. Just a second

…sions

Also contains a run of `pre-commit autoupdate`, and a bump of cibuildwheel
to its latest tag for CPython 3.13 support.

But, since we build for 3.10+ with SABI from 3.12 onwards, we don't even
need a dedicated Python 3.13 build job or toolchain - the wheels from 3.12
can be reused.

Simplifies some version-dependent logic around assembling the bazel
build command in setup.py, and fixes a possible unbound local error in
the toolchain patch context manager.
@nicholasjng
Copy link
Contributor Author

I decided to extend the SABI argument to setup.py, so we don't download any 3.13 toolchains anymore (since we're not yet building free-threaded wheels).

The current setup should now stay in place until either a free-threaded wheel is needed, or the minimum required Python is bumped (which is a few years away).

@dmah42 dmah42 merged commit a6af6ee into google:main Nov 6, 2024
88 of 89 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants