-
-
Notifications
You must be signed in to change notification settings - Fork 32.7k
bpo-39432: Implement PEP-489 algorithm for non-ascii "PyInit_*" symbol names in distutils #18150
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
bpo-39432: Implement PEP-489 algorithm for non-ascii "PyInit_*" symbol names in distutils #18150
Conversation
8c9ef9e
to
13b3cbe
Compare
…l names in distutils to make it export the correct init symbol also on Windows.
13b3cbe
to
7cdb5cc
Compare
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.
Ah! I didn't know about this part of the code :(
The change looks good to me, but it would be much better to have a test ensuring that such an extension can actually be imported successfully.
Please keep the bug open for that, if you merge as is.
Thanks @scoder for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8. |
GH-18546 is a backport of this pull request to the 3.8 branch. |
…l names in distutils (GH-18150) (GH-18546) Make it export the correct init symbol also on Windows. https://bugs.python.org/issue39432 (cherry picked from commit 9538bc9) Co-authored-by: Stefan Behnel <stefan_ml@behnel.de>
|
|
This PR is for |
|
Make it export the correct init symbol also on Windows.
https://bugs.python.org/issue39432
Automerge-Triggered-By: @encukou