Bug 682553

Again I got the URLs wrong:

Spec URL:  http://cern.ch/straylen/rpms/python26-pycurl/python26-pycurl.spec
SRPM URL: http://cern.ch/straylen/rpms/python26-pycurl/python26-pycurl-7.15.5.1-9.el5.src.rpm

Hi Steve,
   i did some review on this:
- rpmlint output looks ok for SPEC, SRPM and scratch builds at Fedora's koji.
- the name is according to guidelines
- the spec file has the correct name
- Regarding license the vendor provides the following string:
"PycURL is dual licensed under the LGPL and an MIT/X derivative license based on the cURL license. You can redistribute and/or modify PycURL according to the terms of either license."
I'm not sure if MIT should also be mentioned as dual license.
- Spec file is legible and seems to be in American English.
- MD5 sum of source matches upstream.
- Scratch build doesn't have any issues on any of requested archs (EPEL5):
http://koji.fedoraproject.org/koji/taskinfo?taskID=2893468
- SPEC doesn't handle locales at all so no issue here.
- There is no shared lib in any default dynamic linker's path.
- There is no bundle of any system library
- There is no Prefix header
- All created folders are owned.
- All files are listed once in %files section
- defattr is included in %files section
- macros are used instead of hardcoded names
- Documentation COULD be in -doc subpackage but this is left to the packager. I guess it is better to be consistent with python-pycurl package for same arch (where there is NO -doc package).
- %doc files includes documentation, examples and tests thus it is not required for the program to run properly
- there are no header files or static libraries to use -devel or -static
- there are no library files with a suffix (e.g. libfoo.so.1.1)
- there are no .la files
- there is no GUI application for this package
- the package doesn't own any file or directory that is owned by other packages.
- the filenames of all files in the package are in valid UTF-8

Finally the package from the scratch build installs without any issue in a RHEL5 clone and loads fine:
$ python2.6 
Python 2.6.5 (r265:79063, Jun  4 2010, 21:42:56) 
[GCC 4.1.2 20080704 (Red Hat 4.1.2-48)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import curl
>>> curl
<module 'curl' from '/usr/lib64/python2.6/site-packages/curl/__init__.pyc'>


To sum up i have only the following minor comments:
- Regarding license the vendor provides the following string:
"PycURL is dual licensed under the LGPL and an MIT/X derivative license based on the cURL license. You can redistribute and/or modify PycURL according to the terms of either license."
I'm not sure if MIT should also be mentioned as dual license.
- Documentation COULD be in -doc subpackage but this is left to the packager. I guess it is better to be consistent with python-pycurl package for same arch (where there is NO -doc package).


Also note that i'm not authorized to do an official review so these are just comments to the packager and (helpful?) info for the official reviewer :).

Regards,
Christos

I was also a bit confused by this "MIT" status as well. I found this one the website but did
you find it in the source code at all I did not.

Steve.

Hi again,
   just checked the CVS [1] repository. Dual license is introduced in version 7.18.1. So as long as you are packaging against 7.15.5.1 this should be fine.

Christos

[1] http://pycurl.cvs.sourceforge.net/viewvc/pycurl/pycurl/ChangeLog?revision=1.249&view=markup

I should have commented, I not doing the newest version of pycurl since that is not suitable
for a RHEL5 curl and this version matches the stock python24 version of pycurl in the RHEL5.

Thanks for the informal review.

Steve.

Fedora Review python26-pycurl

rpmlint output:

rpmlint 2248713324973631270127050/result/python26-pycurl/*.rpm
python26-pycurl.src: W: spelling-error Summary(en_US) libcurl -> lib curl, lib-curl, curlicue
python26-pycurl.src: W: spelling-error %description -l en_US libcurl -> lib curl, lib-curl, curlicue
python26-pycurl.src: W: spelling-error %description -l en_US urllib -> gullible
python26-pycurl.src: W: invalid-url Source0: http://pycurl.sourceforge.net/download/pycurl-7.15.5.1.tar.gz HTTP Error 404: Not Found
python26-pycurl.x86_64: W: spelling-error Summary(en_US) libcurl -> lib curl, lib-curl, curlicue
python26-pycurl.x86_64: W: spelling-error %description -l en_US libcurl -> lib curl, lib-curl, curlicue
python26-pycurl.x86_64: W: spelling-error %description -l en_US urllib -> gullible
python26-pycurl.x86_64: E: incorrect-fsf-address /usr/share/doc/python26-pycurl-7.15.5.1/COPYING
3 packages and 0 specfiles checked; 1 errors, 7 warnings.

The source tarball has moved:
http://pycurl.sourceforge.net/download/00-OLD-VERSIONS/pycurl-7.15.5.1.tar.gz


+ Package name is appropriate for a python 2.6 package
+ Specfile is named after the package
+ Package is licensed under a Fedoara approved license
+ Package license matches the license statements in the sources
+ License file (COPYING) is included in the package
+ Specfile is written in legible English
+ Source in the package matches the upstream source:

md5sum pycurl-7.15.5.1.tar.gz srpm/pycurl-7.15.5.1.tar.gz 
464cfbeba150d99d92a407c7c8b751de  pycurl-7.15.5.1.tar.gz
464cfbeba150d99d92a407c7c8b751de  srpm/pycurl-7.15.5.1.tar.gz

+ Package builds in mock
+ Build requirements are sane
+ No locales
+ No shared libraries in default library path
+ No bundled libraries
+ Package owns directories it created
+ No duplicates in %files
+ File permissions are sane
+ The specfile uses macros consistently
+ Package contains code
+ %doc not runtime essential
+ No static libraries
+ No development files
+ No subpackages
+ No libtool archives
+ Package does not own other's directories
+ Installed filenames are UTF-8


Apart from that the source has moved to a new location on the upstream server, the package is good. Approved.

Steve, do you plan to submit a SCM request for this anytime soon?  Anything I can do to help?

Hi

I missed this had been approved. I can do the submission next week.

Steve.

New Package SCM Request
=======================
Package Name: python26-curl
Short Description: A Python interface to libcurl
Owners: stevetraylen
Branches: el5

Did you want python26-curl or python26-pycurl?  The SCM request says one, the
ticket summary and the contents of the review aseem to say the other.

New Package SCM Request
=======================
Package Name: python26-pycurl
Short Description: A Python interface to libcurl
Owners: stevetraylen
Branches: el5


Indeed , thanks for checking. It should be python26-pycurl to match
the existing 'pycurl' package name.

Many Thanks.

Git done (by process-git-requests).

python26-pycurl-7.15.5.1-9.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/python26-pycurl-7.15.5.1-9.el5

python26-pycurl-7.15.5.1-9.el5 has been pushed to the Fedora EPEL 5 testing repository.

python26-pycurl-7.15.5.1-9.el5 has been pushed to the Fedora EPEL 5 stable repository.