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

python: Switch to sysconfig module #554

Merged
merged 1 commit into from Feb 6, 2022

Conversation

jdemel
Copy link
Contributor

@jdemel jdemel commented Jan 7, 2022

The distutils module is deprecated and will be removed in Python 3.12.
We follow GNU Radio's example as well as the recommendations in PEP 632
and switch from distutils.sysconfig to sysconfig. Along with that
change comes one fewer package dependency on Debian. We no longer need
python3-distutils as well as an improvement to better distinguish
between dist-packages and site-packages.

UPDATE
A few more references to this issue.
PEP632 that started the deprecation process.
A discussion on site vs dist packages on Debian that we hope to fix too.

UPDATE 2
Python sysconfig was introduced in Python 3.2. We rely on Python 3.4+ and thus, we can assume sysconfig is always present.

The `distutils` module is deprecated and will be removed in Python 3.12.
We follow GNU Radio's example as well as the recommendations in PEP 632
and switch from `distutils.sysconfig` to `sysconfig`. Along with that
change comes one fewer package dependency on Debian. We no longer need
`python3-distutils` as well as an improvement to better distinguish
between `dist-packages` and `site-packages`.

Signed-off-by: Johannes Demel <demel@uni-bremen.de>
Copy link
Contributor

@michaelld michaelld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@michaelld michaelld merged commit 079f39f into gnuradio:main Feb 6, 2022
@jdemel jdemel deleted the update-python-packaging branch July 1, 2022 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants