Spec URL: http://cern.ch/straylen/python26-pycurl.spec SRPM URL: http://cern.ch/straylen/python26-pycurl-7.15.5.1-9.el5.src.rpm Description: PycURL is a Python interface to libcurl. PycURL can be used to fetch objects identified by a URL from a Python program, similar to the urllib Python module. PycURL is mature, very fast, and supports a lot of features. This is an EPEL5 only package. Steve.
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.