-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
[WIP] gh-92584: Rename the distutils package to _distutils #92585
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
Conversation
Rename Lib/distutils/ directory to Lib/_distutils/.
Keep _distutils.tests.support for peg_generator tests
Replace distutils.spawn.find_executable() with shutil.which().
test_tools and test_venv fail because of pip: pypa/pip#11103 |
By default, Temporary workaround: |
For now, this PR is a draft, and so I didn't write any documentation (no NEWS entry, no What's New in Python 3.12, etc.). |
|
Related issue: #92584 |
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 is correct, and seems like the best approach. We just have to fix pip.
@erlend-aasland and I will have the last modules ported to |
See the issue #92584: peg_generator, c-analyzer tool and test_cppext still use distutils. I would prefer to not break them. |
I modified test_cppext to build the C++ extension with setuptools. So far, so good: it works on all buildbots. We can try a similar approach for test_peg_generator. Right now, test_peg_generator builds a library to make the test faster. I don't know how to port distutils code to setuptools. |
How is it going? Is this work tracked somewhere? For example, configure.ac mentions the array module, but setup.py still has code to build this extension. Is there a plan to remove the setup.py code, if the configure/Makefile code is enough? Or is there still a reason to keep the setup.py code? |
I have created GH-93939 tl;dr every module that uses |
It seems like most core devs don't want to keep distutils hidden in the stdlib, but want to directly remove it. So I close my PR. Remaining steps are tracked by other issues. |
No description provided.