Bug 1479813

Summary: hamster-time-tracker: Missing build-time dependency on python2-devel
Product: [Fedora] Fedora Reporter: Iryna Shcherbina <ishcherb>
Component: hamster-time-trackerAssignee: Ankur Sinha (FranciscoD) <sanjay.ankur>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 27CC: fschwarz, sanjay.ankur, torsava
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: hamster-time-tracker-2.0-0.12.rc1.fc28 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-08-16 14:47:23 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 12:53:44 UTC
Created attachment 1311199 [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 hamster-time-tracker-2.0-0.11.rc1.fc27:

$ dnf repoquery --disablerepo=* --enablerepo=rawhide --requires hamster-time-tracker | grep python
dbus-python
gnome-python2-gconf
gnome-python2-libwnck
python-gobject


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. Please review it and apply.

[0] https://fedoraproject.org/wiki/Packaging:Python#Dependencies

Comment 1 Jan Kurik 2017-08-15 07:31:17 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:10:04 UTC
Also opened a Pagure PR with the patch: https://src.fedoraproject.org/rpms/hamster-time-tracker/pull-request/1

Comment 3 Tomas Orsava 2017-08-16 14:47:23 UTC
I merged the PR.