Bug 1281992

Summary: [PATCH] Update to python3
Product: [Fedora] Fedora Reporter: Ralph Bean <rbean>
Component: python-shoveAssignee: Tomas Orsava <torsava>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 24CC: cstratak, dkrejci, lmacken, pviktori, torsava
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: python-shove-0.6.4-3.fc25 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-05-25 16:12:09 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:
Bug Depends On: 1281998    
Bug Blocks: 1285816, 1333770    
Attachments:
Description Flags
Patch to add python3 support.
none
Patch to add python3 support.
none
A new rendition of a Python 3 patch none

Description Ralph Bean 2015-11-14 01:13:13 UTC
Created attachment 1093962 [details]
Patch to add python3 support.

Upstream supports it.  We should do it.  :)

This also updates the python macros to the new ones in the packaging guidelines.

It also adds a new dep on python-stuf, which is apparently required but was missing.

Comment 1 Jan Kurik 2016-02-24 13:57:12 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 24 development cycle.
Changing version to '24'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora24#Rawhide_Rebase

Comment 2 Charalampos Stratakis 2016-03-15 11:38:18 UTC
Created attachment 1136560 [details]
Patch to add python3 support.

Modified Ralph Bean's patch so it can be applied to the latest build.

Scratch builds for rawhide and F24:

rawhide:  http://koji.fedoraproject.org/koji/taskinfo?taskID=13353073
f24:      http://koji.fedoraproject.org/koji/taskinfo?taskID=13353161

Comment 3 Charalampos Stratakis 2016-03-15 11:40:21 UTC
Maintainer is it ok to push the changes?

Comment 4 Ralph Bean 2016-03-29 01:40:57 UTC
Could be low hanging fruit to fix this one, Luke.

Comment 5 Tomas Orsava 2016-05-06 15:17:12 UTC
Hi Luke,
as this is a bit of a time sensitive issue, we would really appreciate if you could take a look at the patch as soon as possible.
If you are time constrained right now, I hope you won't mind if we do the review and push the changes ourselves in case you won't respond within a week.

Thank you!

Comment 6 Tomas Orsava 2016-05-11 12:42:21 UTC
Created attachment 1156131 [details]
A new rendition of a Python 3 patch

So, to go with the trend I also modified the existing (Charalampos') patch. I fixed a name of one dependency and created a macro for the long description so it can be provided in full with each subpackage, as god and the packager intended.

There is a slight problem that the package python2/3-stuf which is a runtime dependency of this package has a bug, which I promptly reported [0], therefore it is not possible to test this package to the full extent until the bug is fixed. 

Nevertheless, I believe the patch is solid and can be pushed, as the packager has not responded in half a year. I hope you won't mind, Luke.

[0] https://bugzilla.redhat.com/show_bug.cgi?id=1335144

Comment 7 Tomas Orsava 2016-05-11 12:50:48 UTC
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=14009308