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
PEP 632: Rename the distutils package to _distutils #92584
Comments
Rename Lib/distutils/ directory to Lib/_distutils/.
I reported this issue to pip: pypa/pip#11103 |
The distutils package was deprecated by the issue #85454. |
PEP 632 "Deprecate distutils module": https://peps.python.org/pep-0632/ |
That seems like a good plan. Thanks. |
My fear with renaming it is that it lingers indefinitely, and diverging in subtle ways from pypa/distutils. Why not address the dependencies and fully remove the functionality? Regardless, I agree renaming is preferable to not removing the module at all. |
Someone has to modify Python to no longer depend on distutils. The issue description lists code using it. It doesn't seem trivial. |
Would it be reasonable to keep it around in the repo while we don't drop the dependency but not ship it as part of Python installations? |
With my PR, the |
Do note that we have buildbots that run from an installed Python. The tests would need to also be modified to fail gracefully |
I propose skipping tests relying on _distutils if _distutils is not available. |
Rewrite test_cppext to run in a virtual environment and to build the C++ extension with setuptools rather than distutils.
test_decimal now uses shutil.which() rather than deprecated distutils.spawn.find_executable().
test_decimal now uses shutil.which() rather than deprecated distutils.spawn.find_executable().
The distutils module was deprecated in Python 3.10 and is scheduled for removal in Python 3.12.
Right now, Python still uses distutils for different purpose:
Tools/c-analyzer/
uses distutils to get a C preprocessor.Until Python can be built and used without distutils, I propose to first rename the
distutils
package to_distutils
: rename thLib/distutils/
directory toLib/_distutils/
.Right now, test_venv fails without distutils, because pip fails on importing the distutils module in
pip/_internal/locations/_distutils.py
: https://github.com/pypa/pip/blob/cb24fb4052ca8ab8009866b0de61980c81a7e13c/src/pip/_internal/locations/_distutils.py#L9-L12I created the PR #92585 which renames distutils to _distutils.
The text was updated successfully, but these errors were encountered: