-
-
Notifications
You must be signed in to change notification settings - Fork 29.3k
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-38597: Never statically link extension initialization code on Windows #18724
Conversation
Codecov Report
@@ Coverage Diff @@
## master #18724 +/- ##
===========================================
+ Coverage 82.13% 83.23% +1.09%
===========================================
Files 1955 1571 -384
Lines 584601 415210 -169391
Branches 44484 44484
===========================================
- Hits 480156 345590 -134566
+ Misses 94793 59972 -34821
+ Partials 9652 9648 -4
Continue to review full report at Codecov.
|
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 looks basically fine to me, but I'm not familiar enough to really say yay or nay. Paging @eryksun :)
Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8. |
…dows (pythonGH-18724) (cherry picked from commit ce3a498) Co-authored-by: Steve Dower <steve.dower@python.org>
GH-18758 is a backport of this pull request to the 3.8 branch. |
Sorry, @zooba, I could not cleanly backport this to |
GH-18759 is a backport of this pull request to the 3.7 branch. |
https://bugs.python.org/issue38597