Bug 1497141 - lorax: lorax requires both Python 2 and Python 3
Summary: lorax: lorax requires both Python 2 and Python 3
Alias: None
Product: Fedora
Classification: Fedora
Component: lorax
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Brian Lane
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2017-09-29 09:54 UTC by Iryna Shcherbina
Modified: 2018-01-29 16:40 UTC (History)
7 users (show)

Fixed In Version: lorax-28.0-1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2018-01-29 16:40:55 UTC
Type: Bug

Attachments (Terms of Use)

Description Iryna Shcherbina 2017-09-29 09:54:07 UTC
The lorax RPM requires both Python 2 and Python 3.

Except in very special circumstances, there is no need for one package
to drag in both Python stacks. Usually, this is a packaging error: for
example, a stray "/usr/bin/python" shebang in a Python 3 package can
introduce a Python 2 dependency.

Please split your package, or remove the stray dependencies.
There is a section on shebangs in the Python RPM Porting Guide [0]
which covers this issue.

It's ok to do this in Rawhide only, however, it would be greatly
appreciated if you could push it to Fedora 27 as well.

If anything is unclear, or if you need any kind of assistance, you can
ask on IRC (#fedora-python on Freenode), or reply here. We'll be happy
to help investigating or fixing this issue!

[0] http://python-rpm-porting.readthedocs.io/en/latest/application-modules.html#are-shebangs-dragging-you-down-to-python-2

Comment 1 Iryna Shcherbina 2017-10-11 16:40:11 UTC
I have prepared a Pagure PR with the fix [0] for the issue and would kindly ask to review it and merge at will.

[0] https://src.fedoraproject.org/rpms/lorax/pull-request/1

Comment 2 Brian Lane 2017-10-11 17:32:26 UTC
Thanks, but running sed in the spec isn't the right way to fix this.

PR here - https://github.com/rhinstaller/lorax/pull/256

Comment 3 Iryna Shcherbina 2017-10-12 08:44:01 UTC
Hi Brian, fixing this upstream is even better. Thank you for taking a look and opening an upstream PR.

Comment 4 Fedora Blocker Bugs Application 2017-10-12 15:25:13 UTC
Proposed as a Freeze Exception for 27-beta by Fedora user bcl using the blocker tracking app because:

 Would be nice not to pull in python2

Comment 5 Brian Lane 2017-10-17 17:29:20 UTC
Fix pushed to rawhide only.

Note You need to log in before you can comment on or make changes to this bug.