-
Notifications
You must be signed in to change notification settings - Fork 963
Disable limited API for free-threaded Python #1051
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 4.x
Are you sure you want to change the base?
Conversation
ea2620b to
8d3e661
Compare
|
With this change, I was able to build a wheel for free-threaded Python with both Is there any additional information I can provide to help move this forward? |
|
hello, how to run the build process |
This PR allows installing the library for free-threaded Python the same way as any version. This can be done like this: I can confirm that it works on x86_64 Linux but I haven't tried on other platforms. |
|
Hi @sergregory, just checking in. Is there any more information I can provide to help move this forward? Thanks! |
rgommers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is correct, and merging it as is will prevent a build failure. By itself this PR doesn't make free-threaded Python supported but it's helpful to unblock other improvements, and cannot hurt. I recommend hitting the green button here.
|
@asmorkalov sorry for the direct ping, but it would be really helpful if you could merge this one-liner fix - it avoids a build failure, and makes it possible to install directly from the default branch (or an sdist after the next release) in the absence of wheels for free-threaded Python. |
Free-threaded Python does not support the Limited C API.1
This PR should make it possible to build OpenCV wheels for Python 3.13t (#1029).
Footnotes
https://docs.python.org/3/howto/free-threading-extensions.html#limited-c-api-and-stable-abi ↩