Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 197476 - Review Request: python-cvstoys
Summary: Review Request: python-cvstoys
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jochen Schmitt
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-07-02 19:49 UTC by Michael J Knox
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-06 19:32:01 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michael J Knox 2006-07-02 19:49:15 UTC
Spec URL: http://www.knox.net.nz/~michael/python-cvstoys.spec
SRPM URL: http://www.knox.net.nz/~michael/python-cvstoys-1.0.10-1.src.rpm

Description: 

CVSToys is a collection of companion tools for the Concurrent Versions System,
better known as CVS.  The biggest best and only toy we've got is Fresh CVS,
a commit monitoring daemon which can keep a mirror of the repository or
checked-out working copy up to date, and send out notifications by e-mail, on
the web as HTML or RSS, or to a chat service such as IRC.

Comment 1 Jochen Schmitt 2006-07-03 17:45:06 UTC
Good:
+ Tarball in SRPM match with upstream.
+ Local build works fine.
+ License OK.
+ rpmlint of source RPM ok.
+ Mock build works fine on FC-5 as root.

Bad:
- No verbatin license in %doc stanza
- I think the Group should be Development/Tools
- Why you set CFLAGS, when no binary file will be created
- rpmlint have the following complaints to the binary RPM.
E: python-cvstoys non-executable-script 
/usr/lib/python2.4/site-packages/cvstoys/setup.py 0644
E: python-cvstoys non-executable-script
/usr/lib/python2.4/site-packages/cvstoys/loginfo.py 0644
E: python-cvstoys non-executable-script
/usr/lib/python2.4/site-packages/cvstoys/clients/words.py 0644

Comment 2 Michael J Knox 2006-07-03 23:28:43 UTC
Updated:

Spec URL: http://www.knox.net.nz/~michael/python-cvstoys.spec
SRPM URL: http://www.knox.net.nz/~michael/python-cvstoys-1.0.10-2.src.rpm

There is no license file provided, upstream has been contacted. 

As for the rpmlint errors, I check other python packages and they were the same. 

Comment 3 Paul Howarth 2006-07-04 06:11:58 UTC
(In reply to comment #2)
> As for the rpmlint errors, I check other python packages and they were the same. 

However, they should be fixable by editing out the shellbangs from the files
concerned, which would shut rpmlint up and make everyone happy :-)

Comment 4 Michael J Knox 2006-07-04 06:36:52 UTC
is that the correct way to fix it? this certainly isn't happening with existing
python-* packages. 

Comment 5 Paul Howarth 2006-07-04 07:38:03 UTC
(In reply to comment #4)
> is that the correct way to fix it?

Yes. The shellbangs are put there by the upstream developers so that they can
run individual parts of the package on their own for debugging. Given that the
files are not installed with exec permissions, this isn't something that an end
user is going to be needing to do.

> this certainly isn't happening with existing
> python-* packages. 

Perhaps many reviewers are happy to let them go because it's not an important
issue (true in itself), or because they don't know how to fix it, or because
they think the fix clutters the spec file and is therefore worse than the
problem itself? My view is that it's best to shut rpmlint up as much as is
reasonably possible (some things just can't be fixed because the package
wouldn't work if they were), so that rpmlint generates as little output as
possible. The benefit of this is that any new issues that crop up with package
version or rpmlint changes are less likely to be lost amongst a barrage of
unimportant issues.



Comment 6 Michael J Knox 2006-07-04 07:46:31 UTC
OK, I will patch it up shortly. Only asked cuz I didn't know and its not been
covered in the python packaging guidelines :) 

Comment 7 Paul Howarth 2006-07-04 09:17:56 UTC
(In reply to comment #6)
> OK, I will patch it up shortly. Only asked cuz I didn't know and its not been
> covered in the python packaging guidelines :) 

It's not a python-specific thing; it happens quite often with perl modules too.

Comment 8 Michael J Knox 2006-07-05 07:38:41 UTC
OK, patched... rpmlint is rather quiet now :-) 

Updates:

Spec URL: http://www.knox.net.nz/~michael/python-cvstoys.spec
SRPM URL: http://www.knox.net.nz/~michael/python-cvstoys-1.0.10-3.src.rpm

Comment 9 Jochen Schmitt 2006-07-06 15:58:33 UTC
rpmlint is noew quite for the binary rpm.

So I will APPROVE your package.

Comment 10 Michael J Knox 2006-07-06 19:32:01 UTC
awesome. Thank you. In devel now and building. 


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