Is it okay to use /usr/bin/python again?

[Florian Weimer]

Or is it still banned in Fedora?

We have some scripts that are dual Python 2/Python 3, and Fedora tooling
forced us to carry a downstream-only patch to replace /usr/bin/python
with /usr/bin/python3.  I'd like to remove this patch.

Thanks,
Florian

[Jan Drögehoff]

According to the guidelines its still required to change the shebang to python3

https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_shebangs

Jan

Jan 4, 2022 11:09:40 AM Florian Weimer <fweimer(a)redhat.com&gt;:

>  Or is it still banned in Fedora?
>  
>  We have some scripts that are dual Python 2/Python 3, and Fedora tooling
>  forced us to carry a downstream-only patch to replace /usr/bin/python
>  with /usr/bin/python3.  I'd like to remove this patch.
>  
>  Thanks,
>  Florian
>  _______________________________________________
>  devel mailing list -- devel(a)lists.fedoraproject.org
>  To unsubscribe send an email to devel-leave(a)lists.fedoraproject.org
>  Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>  List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>  List Archives:
> https://lists.fedoraproject.org/archives/list/[e-mail]
>  Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure

[Gary Buhrmaster]

On Tue, Jan 4, 2022 at 10:09 AM Florian Weimer <fweimer(a)redhat.com&gt; wrote:
>  Or is it still banned in Fedora?
>
>  We have some scripts that are dual Python 2/Python 3, and Fedora tooling
>  forced us to carry a downstream-only patch to replace /usr/bin/python
>  with /usr/bin/python3.  I'd like to remove this patch. 
Unless your package Requires: python-unversioned-command
I do not believe one could be assured there is a /usr/bin/python
on a target system.

[Miro Hrončok]

On 04. 01. 22 11:09, Florian Weimer wrote:
>  Or is it still banned in Fedora?
>  
>  We have some scripts that are dual Python 2/Python 3, and Fedora tooling
>  forced us to carry a downstream-only patch to replace /usr/bin/python
>  with /usr/bin/python3.  I'd like to remove this patch. 
It is still forbidden:

https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_interp...

Instead of a patch, you can use the %py3_shebang_fix macro.

https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#py3_she...

-- 
Miro Hrončok
-- 
Phone: [phone]
IRC: mhroncok

[Vitaly Zaitsev]

On 04/01/2022 11:09, Florian Weimer wrote:
>  We have some scripts that are dual Python 2/Python 3, and Fedora
> tooling
>  forced us to carry a downstream-only patch to replace /usr/bin/python
>  with /usr/bin/python3.  I'd like to remove this patch. 
It is forbidden.

You should switch your SPEC to modern 201x-era guidelines and all 
shebangs will be handled automatically with the %pyproject_install macro.

-- 
Sincerely,
   Vitaly Zaitsev (vitaly(a)easycoding.org)

[Miro Hrončok]

On 04. 01. 22 13:57, Vitaly Zaitsev via devel wrote:
>  On 04/01/2022 11:09, Florian Weimer wrote:
> > We have some scripts that are dual Python 2/Python 3, and Fedora tooling
> > forced us to carry a downstream-only patch to replace /usr/bin/python
> > with /usr/bin/python3.  I'd like to remove this patch.
>  
>  It is forbidden.
>  
>  You should switch your SPEC to modern 201x-era guidelines 
202x-era

>  and all shebangs will 
>  be handled automatically with the %pyproject_install macro. 
That is only happening for files installed to /usr/bin, not to files you use 
during builds.

-- 
Miro Hrončok
-- 
Phone: [phone]
IRC: mhroncok

[Nico Kadel-Garcia]

On Tue, Jan 4, 2022 at 8:11 AM Miro Hrončok <mhroncok(a)redhat.com&gt; wrote:
>  On 04. 01. 22 13:57, Vitaly Zaitsev via devel wrote:
>  > On 04/01/2022 11:09, Florian Weimer wrote:
>  >> We have some scripts that are dual Python 2/Python 3, and Fedora tooling
>  >> forced us to carry a downstream-only patch to replace /usr/bin/python
>  >> with /usr/bin/python3.  I'd like to remove this patch.
>  >
>  > It is forbidden.
>  >
>  > You should switch your SPEC to modern 201x-era guidelines
>
>  202x-era
>
>  > and all shebangs will
>  > be handled automatically with the %pyproject_install macro.
>
>  That is only happening for files installed to /usr/bin, not to files you use
>  during builds. 
It confuses the heck out of building Fedora. "/ur/bin/python" may not
be python3 on a remote system, especially a legacy system. Please,
explicitly use "#!/usr/bin/python3" or "#!/usr/bin/env python3".

[Vitaly Zaitsev]

On 04/01/2022 19:36, Nico Kadel-Garcia wrote:
>  #!/usr/bin/env python3 
Forbidden by Python guidelines too.

-- 
Sincerely,
   Vitaly Zaitsev (vitaly(a)easycoding.org)

[Nico Kadel-Garcia]

On Wed, Jan 5, 2022 at 2:34 AM Vitaly Zaitsev via devel
<devel(a)lists.fedoraproject.org&gt; wrote:
>  On 04/01/2022 19:36, Nico Kadel-Garcia wrote:
>  > #!/usr/bin/env python3
>
>  Forbidden by Python guidelines too. 
Can you point to where that is forbidden? Because it remains quite
popular for various working environments with multiple python
releases, such as RHEL 7 supported through EPEL.

[Vitaly Zaitsev]

On 05/01/2022 08:46, Nico Kadel-Garcia wrote:
>  Can you point to where that is forbidden? 
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_shebang_lines

>  env, /bin/env and /usr/bin/env MUST NOT be used. The interpreter used
> to run packaged applications cannot depend upon what the user has in their personal $PATH.
-- 
Sincerely,
   Vitaly Zaitsev (vitaly(a)easycoding.org)