Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-carrot.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-carrot-0.10.5-1.fc13.src.rpm Project URL: http://pypi.python.org/pypi/carrot Description: carrot is an AMQP messaging queue framework. AMQP is the Advanced Message Queuing Protocol, an open standard protocol for message orientation, queuing, routing, reliability and security. The aim of carrot is to make messaging in Python as easy as possible by providing a high-level interface for producing and consuming messages. At the same time it is a goal to re-use what is already available as much as possible. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2291916 rpmlint output: [fab@laptop011 SRPMS]$ rpmlint python-carrot-0.10.5-1.fc13.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [fab@laptop011 noarch]$ rpmlint python-carrot-0.10.5-1.fc13.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
* Mon Nov 01 2010 Fabian Affolter <fabian> - 0.10.7-1 - Updated to new upstream release 0.10.7 Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-carrot.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-carrot-0.10.7-1.fc14.src.rpm
Created attachment 469111 [details] informal review
Hi, As I'm hoping to become a packager I've done an informal review of this package. Full comments on the ReviewGuidelines are attached, but a couple of things I'd like to highlight are: * no CFLAGS in the %build section -- see http://fedoraproject.org/wiki/PackagingGuidelines#Compiler_flags CFLAGS="$RPM_OPT_FLAGS" %{__python} setup.py build * your dependency on 'Requires: python-json' means I couldn't install it on a stock F14 laptop -- which json implementation do you need? * rpmlint threw warnings about no buildroot. BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the default that comes with rpmdev-newspec * finally upstream says on https://github.com/ask/carrot/#readme that "Carrot is discontinued in favor of the new Kombu framework. Kombu is ready, start to use it now! Kombu comes with a Carrot compatible API, so it's easy to port your software. Carrot will not be actively maintained, only critical bugs will be fixed." -- you may want to mention this in the description...
Andrew, please don't attach package reviews; just paste them in. Otherwise it is difficult for us to see them.
OK, Will do. Wasn't sure of the ettiquette of pasting large chunks of text in.
Fabian: Andrew's comments appear to be correct wrt to python-json - python2.6 (and 3000) contains a json module, so that BR should probably be stripped (or conditionalized if you plan on building this for the EL < 6) If you do plan on building for EL < 6 you need Buildroot specified. Andrew's informal review looks pretty good
I'll ACK Andrew's review as well.
Fabian, are you still interested in submitting this to Fedora? If not, please reply so we can have someone else work on this, since python-carrot is a dep for openstack-nova. Thanks.
This hasn't been touched since December 10th - despite four additional people providing comments and bpepple has indicated that he's interested in picking it up. http://fedoraproject.org/wiki/PackageMaintainers/Policy#Submitter_not_responding
*** This bug has been marked as a duplicate of bug 673637 ***