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

[Petr Lautrbach <plautrba@xxxxxxxxxx>]

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>

> ---
>  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