mbox series

[0/5,v3] Remove dependency on the Python module distutils

Message ID 20221104143616.1089636-1-jwcart2@gmail.com (mailing list archive)
Headers show
Series Remove dependency on the Python module distutils | expand

Message

James Carter Nov. 4, 2022, 2:36 p.m. UTC
The distutils package is deprecated and scheduled to be removed in
Python 3.12.

When building the SELinux userspace we currently get warnings like the following:
<string>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
<string>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives

I am not a Python expert, but everything seems to work with these patches.
I would appreciate other people testing this and any Python experts to let
me know if this is the right way of replacing distutils.

v3: Removed a second usage of distutils in python/sepolicy/sepolicy/gui.py

v2: Use sysconfig.get_path('purelib'... when original used
    get_python_lib(prefix=... and use sysconfig.get_path('platlib'...
    when original used get_python_lib(plat_specific=1, prefix=...

James Carter (5):
  libselinux: Remove dependency on the Python module distutils
  libsemanage: Remove dependency on the Python module distutils
  python: Remove dependency on the Python module distutils
  scripts: Remove dependency on the Python module distutils
  README.md: Remove mention of python3-distutils dependency

 README.md                             | 1 -
 libselinux/src/Makefile               | 2 +-
 libselinux/src/setup.py               | 2 +-
 libsemanage/src/Makefile              | 2 +-
 python/semanage/Makefile              | 2 +-
 python/sepolgen/src/sepolgen/Makefile | 2 +-
 python/sepolicy/sepolicy/gui.py       | 4 ++--
 python/sepolicy/setup.py              | 2 +-
 scripts/env_use_destdir               | 2 +-
 scripts/run-scan-build                | 2 +-
 10 files changed, 10 insertions(+), 11 deletions(-)

Comments

Petr Lautrbach Nov. 7, 2022, 8:13 a.m. UTC | #1
James Carter <jwcart2@gmail.com> writes:

> The distutils package is deprecated and scheduled to be removed in
> Python 3.12.
>
> When building the SELinux userspace we currently get warnings like the following:
> <string>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
> <string>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
>
> I am not a Python expert, but everything seems to work with these patches.
> I would appreciate other people testing this and any Python experts to let
> me know if this is the right way of replacing distutils.
>
> v3: Removed a second usage of distutils in python/sepolicy/sepolicy/gui.py
>
> v2: Use sysconfig.get_path('purelib'... when original used
>     get_python_lib(prefix=... and use sysconfig.get_path('platlib'...
>     when original used get_python_lib(plat_specific=1, prefix=...
>
> James Carter (5):
>   libselinux: Remove dependency on the Python module distutils
>   libsemanage: Remove dependency on the Python module distutils
>   python: Remove dependency on the Python module distutils
>   scripts: Remove dependency on the Python module distutils
>   README.md: Remove mention of python3-distutils dependency

Acked-by: Petr Lautrbach <plautrba@redhat.com>


>  README.md                             | 1 -
>  libselinux/src/Makefile               | 2 +-
>  libselinux/src/setup.py               | 2 +-
>  libsemanage/src/Makefile              | 2 +-
>  python/semanage/Makefile              | 2 +-
>  python/sepolgen/src/sepolgen/Makefile | 2 +-
>  python/sepolicy/sepolicy/gui.py       | 4 ++--
>  python/sepolicy/setup.py              | 2 +-
>  scripts/env_use_destdir               | 2 +-
>  scripts/run-scan-build                | 2 +-
>  10 files changed, 10 insertions(+), 11 deletions(-)
>
> -- 
> 2.38.1
James Carter Nov. 9, 2022, 1:47 p.m. UTC | #2
On Mon, Nov 7, 2022 at 3:14 AM Petr Lautrbach <plautrba@redhat.com> wrote:
>
> James Carter <jwcart2@gmail.com> writes:
>
> > The distutils package is deprecated and scheduled to be removed in
> > Python 3.12.
> >
> > When building the SELinux userspace we currently get warnings like the following:
> > <string>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
> > <string>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
> >
> > I am not a Python expert, but everything seems to work with these patches.
> > I would appreciate other people testing this and any Python experts to let
> > me know if this is the right way of replacing distutils.
> >
> > v3: Removed a second usage of distutils in python/sepolicy/sepolicy/gui.py
> >
> > v2: Use sysconfig.get_path('purelib'... when original used
> >     get_python_lib(prefix=... and use sysconfig.get_path('platlib'...
> >     when original used get_python_lib(plat_specific=1, prefix=...
> >
> > James Carter (5):
> >   libselinux: Remove dependency on the Python module distutils
> >   libsemanage: Remove dependency on the Python module distutils
> >   python: Remove dependency on the Python module distutils
> >   scripts: Remove dependency on the Python module distutils
> >   README.md: Remove mention of python3-distutils dependency
>
> Acked-by: Petr Lautrbach <plautrba@redhat.com>
>

This series has been merged.
Jim

>
> >  README.md                             | 1 -
> >  libselinux/src/Makefile               | 2 +-
> >  libselinux/src/setup.py               | 2 +-
> >  libsemanage/src/Makefile              | 2 +-
> >  python/semanage/Makefile              | 2 +-
> >  python/sepolgen/src/sepolgen/Makefile | 2 +-
> >  python/sepolicy/sepolicy/gui.py       | 4 ++--
> >  python/sepolicy/setup.py              | 2 +-
> >  scripts/env_use_destdir               | 2 +-
> >  scripts/run-scan-build                | 2 +-
> >  10 files changed, 10 insertions(+), 11 deletions(-)
> >
> > --
> > 2.38.1
>