Bug 664054

Summary: Review Request: python-rhsm - Python library to communicate with a Red Hat Unified Entitlement Platform
Product: [Fedora] Fedora Reporter: Devan Goodwin <dgoodwin>
Component: Package ReviewAssignee: Jordan OMara <jomara>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: athomas, bkearney, fedora-package-review, jesusr, jomara, notting
Target Milestone: ---Flags: jomara: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-12-21 19:26:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Devan Goodwin 2010-12-17 20:55:40 UTC
Spec URL: http://dgoodwin.fedorapeople.org/python-rhsm/python-rhsm.spec
SRPM URL: http://dgoodwin.fedorapeople.org/python-rhsm/python-rhsm-0.94.8-1.fc14.src.rpm
Description: Small Python library for communicating with a Red Hat Unified Entitlement Platform. (see the Candlepin project: https://fedorahosted.org/candlepin/)

Comment 1 Jordan OMara 2010-12-17 21:55:11 UTC
rpmlint

[jomara@jomara-host SRPMS]$ rpmlint python-rhsm-0.94.8-1.fc14.src.rpm 
python-rhsm.src: W: invalid-url Source0: python-rhsm-0.94.8.tar.gz
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

This is because that tarball isn't hosted at fedorahosted.org. Please provide instructions on how to get the tarball with git or a direct link, so I can md5sum the provided and hosted sources.

MUST ITEMS: 

package name : check
spec name : check
standard packaging guidelines : check
python packaging guidelines : 
  macro definitions for RHEL5 & below? not a showstopper
  otherwise check
license : check
english spec : check
legible spec : check
upstream src md5sum check : [FAIL] - see above
build check : check
build requires : check
locales : check
binary rpm : check
system library bundle : check
directory ownership: check
macros : check
doc : check
la libtools : check
other package file ownership : check
utf-8 filenames : check

SHOULD items that are missing:
query upstream license file
extra language translations


Summary: Please provide a location to download the source, and I think you are good to go.

Comment 2 Jesus M. Rodriguez 2010-12-17 22:41:30 UTC
git clone git://git.fedorahosted.org/git/candlepin.git/
cd client/python-rhsm
tito build --tag python-rhsm-%{name}-%{version}-%{release} --tgz | grep "Wrote:" | awk '{print $2}' | xargs -n 1 md5sum

I will add this comment to the spec file.

Comment 3 Jesus M. Rodriguez 2010-12-17 22:51:34 UTC
added comment on how to generate the tarball to the spec file. I left out
the part on how to get the md5sum as that's only useful for a reviewer.

Here are the updated srpm and spec file.

Spec URL: http://jmrodri.fedorapeople.org/python-rhsm.spec
SRPM URL:
http://jmrodri.fedorapeople.org/python-rhsm-0.94.9-1.fc13.src.rpm

Comment 4 Jordan OMara 2010-12-20 13:39:15 UTC
Looks good! One extremely pedantic comment: the tito build RPM command ( tito build --tag python-rhsm-%{name}-%{version}-%{release} --tgz) would end up building python-rhsm-python-rhsm-0.94.9-1.

What's important is that I was able to cleanly build a source archive from the upstream repo, and it's MD5sum matches what you provided in the SRPM. Setting +

Comment 5 Jesus M. Rodriguez 2010-12-20 18:02:45 UTC
(In reply to comment #4)
> Looks good! One extremely pedantic comment: the tito build RPM command ( tito
> build --tag python-rhsm-%{name}-%{version}-%{release} --tgz) would end up
> building python-rhsm-python-rhsm-0.94.9-1.
> 
> What's important is that I was able to cleanly build a source archive from the
> upstream repo, and it's MD5sum matches what you provided in the SRPM. Setting +

Oops my typo. Sorry.

Comment 6 Devan Goodwin 2010-12-20 18:36:16 UTC
New Package SCM Request
=======================
Package Name: python-rhsm
Short Description: Small Python library for communicating with a Red Hat Unified
Entitlement Platform.
Owners: dgoodwin jmrodri 
Branches: f14 el6
InitialCC:

Comment 7 Kevin Fenzi 2010-12-21 06:26:36 UTC
Git done (by process-git-requests).

Comment 8 Bryan Kearney 2011-06-09 13:33:55 UTC
Package Change Request
======================
Package Name: python-rhsm
New Branches: el5 
Owners: dgoodwin, jmrodri
InitialCC: 

Pulp would like to see this in el5. It is currently in el6.

Comment 9 Gwyn Ciesla 2011-06-09 13:55:40 UTC
Git done (by process-git-requests).