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

distutils: Deprecation warning #1164

Open
acxz opened this issue Feb 11, 2022 · 3 comments · May be fixed by #1165
Open

distutils: Deprecation warning #1164

acxz opened this issue Feb 11, 2022 · 3 comments · May be fixed by #1165

Comments

@acxz
Copy link
Contributor

acxz commented Feb 11, 2022

/opt/ros/noetic/share/catkin/cmake/interrogate_setup_dot_py.py:43: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
  import distutils.core

catkin version: 0.8.10
py version: 3.10.2

@acxz acxz linked a pull request Feb 11, 2022 that will close this issue
@gbiggs
Copy link
Contributor

gbiggs commented Feb 14, 2022

As there are still likely to be a a lot of packages using distutils in Noetic, I do not think it is safe to remove support for it at this time. Python 3.10 has just been released and 3.12 is not yet scheduled, so we still have a long time before it will become unavailable in the most up-to-date version on Python. On top of that, Noetic targets Python 3.8, so it is not required to handle a Python version where distutils has been removed.

A better time to do this is probably in a post-Noetic version of catkin.

@tfoote
Copy link
Member

tfoote commented Feb 15, 2022

Also before removing the support it would be best to provide a deprecation warning to users with a stated interval by which they will need to have migrated forward.

@acxz
Copy link
Contributor Author

acxz commented Feb 24, 2022

packages using distutils in Noetic

Do you know of a smart way to find out which Noetic packages use distutils?

provide a deprecation warning

I can create a PR with a deprecation warning when distutils gets imported. Is there documentation on a proper format to coding this deprecation warning?

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 a pull request may close this issue.

3 participants