-
Notifications
You must be signed in to change notification settings - Fork 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
{Extension}: drop distutils.sysconfig, deprecated #20391
Conversation
Python is breaking backward compatibility by removing distutils.sysconfig without providing an alternative. For now, hard-code the dist-packages path until they provide a different solution.
Extension |
The PR looks good to me.
I think Python does provide an alternative Here is the output from official Python docker image (which is also built on Debian): > docker run -it --rm python:3.10
Python 3.10.0 (default, Nov 17 2021, 15:26:39) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import distutils.sysconfig
<stdin>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
<stdin>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
>>> import sysconfig
>>> distutils.sysconfig.get_python_lib()
'/usr/local/lib/python3.10/site-packages'
>>> sysconfig.get_path('purelib')
'/usr/local/lib/python3.10/site-packages' The results of |
Wait. Hard-coding it will break #12856 which depends on The only solution I can think of is for Debian to support their customized |
ok, closing for now |
With the official Azure CLI MSI package,
With the official Azure CLI ZIP package,
With the official Azure CLI DEB package,
Hard-coding it to |
Python is breaking backward compatibility by removing distutils.sysconfig
without providing an alternative. For now, hard-code the dist-packages
path until they provide a different solution.