Bug 1479861

Summary: python-atfork: Missing build-time dependency on python2-devel
Product: [Fedora] Fedora Reporter: Iryna Shcherbina <ishcherb>
Component: python-atforkAssignee: James Laska <jal233>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 27CC: jal233, torsava
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: python-atfork-0.1.2-13.fc28 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-08-18 10:42:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Add a build-time dependency on python2-devel none

Description Iryna Shcherbina 2017-08-09 15:11:08 UTC 
Created attachment 1311255 [details]
Add a build-time dependency on python2-devel

A build-time dependency on python2-devel is missing.

This is the reason why the dependency on Python itself is not generated for python-atfork-0:0.1.2-12.fc27:

$ dnf repoquery --disablerepo=* --enablerepo=rawhide --requires python-atfork | grep python
<nothing>


According to Fedora Packaging Guidelines for Python [0], packages building for Python 2 need BuildRequires: python2-devel.

I have attached the patch which adds a missing build-time dependency and modernizes the spec file. Please review it and apply.

[0] https://fedoraproject.org/wiki/Packaging:Python#Dependencies
Comment 1 Jan Kurik 2017-08-15 08:25:36 UTC 
This bug appears to have been reported against 'rawhide' during the Fedora 27 development cycle.
Changing version to '27'.
Comment 2 Iryna Shcherbina 2017-08-15 22:23:20 UTC 
Also created a Pagure PR with a patch:

https://src.fedoraproject.org/rpms/python-atfork/pull-request/1
Comment 3 Tomas Orsava 2017-08-16 14:59:28 UTC 
I merged the PR.