Bug 664054 - Review Request: python-rhsm - Python library to communicate with a Red Hat Unified Entitlement Platform
Summary: Review Request: python-rhsm - Python library to communicate with a Red Hat Un...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jordan OMara
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-12-17 20:55 UTC by Devan Goodwin
Modified: 2014-11-09 22:55 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-12-21 19:26:56 UTC
Type: ---
Embargoed:
jomara: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

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).


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