Re: [PATCH 2/2] python: always use python3 in the shebang of programs using setools

[Petr Lautrbach <plautrba@xxxxxxxxxx>]

Petr Lautrbach <plautrba@xxxxxxxxxx> writes:

> Nicolas Iooss <nicolas.iooss@xxxxxxx> writes:
>
> > setools 4.2.0 dropped support for Python 2. On systems where
> > /usr/bin/python is Python 2, several tools are now broken 
> > because of
> > this. Update the shebang of these tools to /usr/bin/python3.
> >
> > For future reference, as semanage/seobject.py, sepolicy and 
> > sepolgen
> > import setools, every program that uses one of these modules 
> > need to be
> > run with Python 3. The following programs do not use any of 
> > these
> > modules so their shebangs have not been modified:
> >
> >     dbus/selinux_server.py
> >     libsemanage/utils/semanage_migrate_store
> >     mcstrans/share/util/mlscolor-test
> >     mcstrans/share/util/mlstrans-test
> >     sandbox/start
> >
> > Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx>
>
> Acked-by: Petr Lautrbach <plautrba@xxxxxxxxxx>

Both patches are merged already.

> > ---
> >  gui/polgengui.py                  | 2 +-
> >  gui/system-config-selinux.py      | 2 +-
> >  python/audit2allow/audit2allow    | 2 +-
> >  python/audit2allow/sepolgen-ifgen | 2 +-
> >  python/chcat/chcat                | 2 +-
> >  python/sepolicy/sepolicy.py       | 2 +-
> >  sandbox/sandbox                   | 2 +-
> >  7 files changed, 7 insertions(+), 7 deletions(-)
> >
> > diff --git a/gui/polgengui.py b/gui/polgengui.py
> > index 1601dbef0e8d..b1cc99372434 100644
> > --- a/gui/polgengui.py
> > +++ b/gui/polgengui.py
> > @@ -1,4 +1,4 @@
> > -#!/usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  #
> >  # polgengui.py - GUI for SELinux Config tool in 
> >  system-config-selinux
> >  #
> > diff --git a/gui/system-config-selinux.py 
> > b/gui/system-config-selinux.py
> > index ce7c74b1333c..c42301b626fc 100644
> > --- a/gui/system-config-selinux.py
> > +++ b/gui/system-config-selinux.py
> > @@ -1,4 +1,4 @@
> > -#!/usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  #
> >  # system-config-selinux.py - GUI for SELinux Config tool in
> > system-config-selinux
> >  #
> > diff --git a/python/audit2allow/audit2allow 
> > b/python/audit2allow/audit2allow
> > index 18fe0a531d02..09b06f664bc4 100644
> > --- a/python/audit2allow/audit2allow
> > +++ b/python/audit2allow/audit2allow
> > @@ -1,4 +1,4 @@
> > -#! /usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  # Authors: Karl MacMillan <kmacmillan@xxxxxxxxxxxxxxxxx>
> >  # Authors: Dan Walsh <dwalsh@xxxxxxxxxx>
> >  #
> > diff --git a/python/audit2allow/sepolgen-ifgen
> > b/python/audit2allow/sepolgen-ifgen
> > index c482f227193f..be2d093be597 100644
> > --- a/python/audit2allow/sepolgen-ifgen
> > +++ b/python/audit2allow/sepolgen-ifgen
> > @@ -1,4 +1,4 @@
> > -#! /usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  #
> >  # Authors: Karl MacMillan <kmacmillan@xxxxxxxxxxxxxxxxx>
> >  #
> > diff --git a/python/chcat/chcat b/python/chcat/chcat
> > index 5bef0073b7a4..ba398684376d 100755
> > --- a/python/chcat/chcat
> > +++ b/python/chcat/chcat
> > @@ -1,4 +1,4 @@
> > -#! /usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  # Copyright (C) 2005 Red Hat
> >  # see file 'COPYING' for use and warranty information
> >  #
> > diff --git a/python/sepolicy/sepolicy.py 
> > b/python/sepolicy/sepolicy.py
> > index 8cc7e8cc5c1c..1934cd8620b0 100755
> > --- a/python/sepolicy/sepolicy.py
> > +++ b/python/sepolicy/sepolicy.py
> > @@ -1,4 +1,4 @@
> > -#! /usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  # Copyright (C) 2012 Red Hat
> >  # AUTHOR: Dan Walsh <dwalsh@xxxxxxxxxx>
> >  # see file 'COPYING' for use and warranty information
> > diff --git a/sandbox/sandbox b/sandbox/sandbox
> > index c07a1d812a13..1dec07acdcb8 100644
> > --- a/sandbox/sandbox
> > +++ b/sandbox/sandbox
> > @@ -1,4 +1,4 @@
> > -#! /usr/bin/python -Es
> > +#!/usr/bin/python3 -Es
> >  # Authors: Dan Walsh <dwalsh@xxxxxxxxxx>
> >  # Authors: Thomas Liu <tliu@xxxxxxxxxxxxxxxxx>
> >  # Authors: Josh Cogliati