Bug 1436803 - python-igraph: python3-igraph is missing Python 3 version of executables
Summary: python-igraph: python3-igraph is missing Python 3 version of executables
Alias: None
Product: Fedora
Classification: Fedora
Component: python-igraph
Version: 27
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Pierre-YvesChibon
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2017-03-28 17:34 UTC by Iryna Shcherbina
Modified: 2017-09-21 10:32 UTC (History)
3 users (show)

Clone Of:
Last Closed: 2017-09-21 10:32:35 UTC

Attachments (Terms of Use)
Fix of missing Python 3 version executables (8.48 KB, patch)
2017-07-21 13:27 UTC, Jan Beran
no flags Details | Diff

Description Iryna Shcherbina 2017-03-28 17:34:25 UTC
The Python 3 subpackage python3-igraph is missing executables available in the Python 2 subpackage.

According to the Fedora Packaging guidelines for Python [0], if a Python package that supports Python 3 installs executables into /usr/bin,  then either both Python versions of the executables should be packaged, or only the Python 3 version, depending on the functionality they provide.
(Note the "should": having Python 2-only executables is permitted, but strongly discouraged.)

Is it intentional for this particular package?

In case it is not intentional, please provide a Python 3 version of executable, or you can just switch to Python 3 if the functionality is the same regardless of the Python version. The guide for porting Python-based RPMs is available here [1] and you may refer to it for more instructions.

It's ok to do this in Rawhide only, however, it would be greatly
appreciated if you could push it to Fedora 26 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 the issue!

[0] https://fedoraproject.org/wiki/Packaging:Python#Avoiding_collisions_between_the_python_2_and_python_3_stacks
[1] http://python-rpm-porting.readthedocs.io/

Comment 1 Jan Beran 2017-07-21 13:27 UTC
Created attachment 1302395 [details]
Fix of missing Python 3 version executables

Comment 2 Jan Beran 2017-07-21 13:32:10 UTC
Hi pingou,

may I ask you to review the updated spec file, and if everything is fine, to make Fedora build? Thank you.

Comment 3 Jan Kurik 2017-08-15 08:27:01 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 27 development cycle.
Changing version to '27'.

Comment 4 Jan Beran 2017-09-18 13:19:30 UTC
Hi Pingou, please, may I ask for your feedback?

Comment 5 Pierre-YvesChibon 2017-09-18 14:13:47 UTC
Hi Jan,

Thanks for your interest, could you make it a pull-request on src.fp.o?

thanks :)

Comment 6 Jan Beran 2017-09-18 16:47:55 UTC
Hi Pingou,

you can see my PR:


Comment 7 Pierre-YvesChibon 2017-09-21 10:32:35 UTC
Alright, merged and is building in rawhide :)

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