Skip to content
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

Distutils refresh #2653

merged 13 commits into from May 4, 2021

Distutils refresh #2653

merged 13 commits into from May 4, 2021


Copy link

@jaraco jaraco commented Apr 24, 2021

pradyunsg and others added 11 commits October 26, 2020 15:39
Convert blockquotes into literal blocks
…and as it would be used, restoring expectation lost in python/cpython#18743. Ref #2417 where this change was a red-herring for another issue.
Compared to /Ox, /O2 has two additional options:

  /GF: Eliminate Duplicate Strings
  /Gy: Enable Function-Level Linking

Co-Authored-By: Nikita Kniazev <>
Only complain if the config target is >= 10.3 and the current target is
< 10.3. The check was originally added to ensure that incompatible
LDSHARED flags are not used, because '-undefined dynamic_lookup' is
used when building for 10.3 and later, and is not supported on older OS
versions[1]. Apart from that, there should be no problem in general
with using an older target.

[1] python/cpython@6b08a40
Copy workflows from jaraco/skeleton. Enables CI.
bpo-41882: Clean up after CCompiler.has_function()
Use /O2 compiler option instead of /Ox for MSVC
Copy link
Member Author

jaraco commented Apr 25, 2021

I'm confused. I only expected this PR to include a handful of these commits. I need to figure out why the whole history of distutils is appearing in the PR.

Copy link
Member Author

jaraco commented Apr 26, 2021

After merging #2657, the commit log is much shorter, just including the recent changes.

@jaraco jaraco merged commit c492876 into main May 4, 2021
@jaraco jaraco deleted the distutils-refresh branch May 4, 2021 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants