Uh oh!
There was an error while loading. Please reload this page.
gh-81057: Vendor a Subset of distutils for the c-analyzer Tool - #102505
Conversation
ericsnowcurrently
commented
Mar 7, 2023
Note that none of the Windows-related support actually implements the preprocessor code the c-analyzer needs. Here's what that might look like, though: elijahr/python-autopxd2#40. |
sobolevn
left a comment
There was a problem hiding this comment.
Note, distutils is also used in peg_generator:
cpython/Tools/peg_generator/pegen/build.py
Lines 52 to 58 in 1f557f9
Will it be convinient to use the single vendored version for both of them?
ericsnowcurrently
commented
Mar 8, 2023
Thanks for pointing this out. I'll take a look. |
AlexWaygood
commented
Mar 8, 2023
See these two previous issues that were opened on this topic: |
ericsnowcurrently
commented
Mar 8, 2023
(See #92584 (comment).) I'm fine with sharing a solution with peg_generator (e.g. a common vendored distutils). In the meantime, I'd like to land this PR. We can remove this vendored distutils when there's a shared solution. |
distutils was removed in November. However, the c-analyzer relies on it. To solve that here, we vendor the parts the tool needs so it can be run against 3.12+. (Also see gh-92584.)