Spec URL: http://remi.fedorapeople.org/libmemcached.spec SRPM URL: http://remi.fedorapeople.org/libmemcached-0.28-1.fc8.src.rpm Mock log: http://remi.fedorapeople.org/libmemcached-mock.log Koji Scratch build : http://koji.fedoraproject.org/koji/taskinfo?taskID=1321061 Description: libmemcached is a C client library to the memcached server (http://danga.com/memcached). It has been designed to be light on memory usage, and provide full access to server side methods. It also implements several command line tools: memcat - Copy the value of a key to standard output. memflush - Flush the contents of your servers. memrm - Remove a key(s) from the server. memstat - Dump the stats of your servers to standard output. memslap - Generate testing loads on a memcached cluster. memcp - Copy files to memcached servers. memerror - Creates human readable messages from libmemcached error codes. --------- libmemcached is required by some memcached client. I will submit php-pecl-memcached for review.
Could not find any issues except license stuff. Could you please check with Brian that all files have BSD license. Seems like the hash files crc.c, hsieh_hash.c jenkins_hash.c md5.c and murmur_hash.c need special attention in this regard.
----------- Mail to Brian : I haven't do a full license review of your code... (shame on me) It's seems that some files are provided by external sources. Have you some information about how do this license apply to your project : crc.c => ?? hsieh_hash.c => http://www.azillionmonkeys.com/qed/weblicense.html jenkins_hash.c => http://burtleburtle.net/bob/hash/index.html md5.c => RSA ?? murmur_hash.c => http://murmurhash.googlepages.com/ is "public domain" At least some of them (RSA notice) should be distributed in the library/RPM documentation (perhaps a COPYING.xxx and in AUTHORS file). ------------ Response from Brian We leave the external copyright in the top. All other code is BSD. If someone needed to chop something out they could with ease.
I have sent a mail to spot to assist.
Looking at those files: crc.c: BSD [OK] hsieh_hash.c: New license [Checking with RH Legal] jenkins_hash.c: Copyright Only (I will interpret "Do NOT use for cryptographic purposes." as advice after the license, rather than license terms, since it conflicts with "You may use this code any way you wish". The upstream for this file should really change that line to say "We do not recommend that you use this code for cryptographic purposes.") [OK] md5.c: BSD [OK] murmur_hash.c: Public Domain [OK] (They really should have a proper declaration for PD on their website, but I'll let it slide) As soon as I hear back on the hsieh licensing, I'll comment.
Red Hat Legal says the hsieh licenses are non-free. Hopefully, you should be able to delete that hsieh_hash.c file in %pre without any difficulty.
@tom : thanks for this. I will work on a build whithout hsieh_hash.c (I don't understand what you mean about %pre, should be deleted in %prep and Makefile patched to not try to use it) Rehards
------------ Response from Brian Thanks for the heads up. I'll look into adding a configure switch to disable it!
I meant %prep, not %pre. :) Sorry!
Upstream have push a new configure with hsieh hash method disabled by default. Here a updated RPM using this patch Spec URL: http://remi.fedorapeople.org/libmemcached.spec SRPM URL: http://remi.fedorapeople.org/libmemcached-0.28-2.fc8.src.rpm
Thanks Remi, patch seems to work. Starting full review. [ x=ok -=dont't apply !=please fix ?=may fix ] MUST [x] rpmlint must be run on every package [x] package must be named according to the Package Naming Guidelines [x] spec file name must match the base package %{name} [-] package must meet the Packaging Guidelines [x] [BSD] package must be licensed with a Fedora approved license [x] license field in the package spec file must match the actual license [x] includes the text of the license(s) in its own file: include in %doc [x] be written in American English [x] spec file for the package be legible [x] sources used to build the package must match the upstream source md5sum: 3e022f51301a73ddfc3df32de5b748b19fd97c65 : ok [x] compile and build into binary rpms on at least one architecture http://koji.fedoraproject.org/koji/taskinfo?taskID=1331611 [-] not successfully compile an architecture: use ExcludeArch [x] all build dependencies must be listed in BuildRequires [-] spec file MUST handle locales properly [x] shared library files not in any default linker paths: ldconfig [-] relocatable package: the packager must state this fact [x] package must own all directories that it creates [x] not any duplicate files in the %files listing [x] permissions on files must be set properly [x] package must have a %clean section, which contains rm -rf %{buildroot} [x] consistently use macros [x] must contain code, or permissable content [-] large docs should go in a -doc subpackage [x] %doc must not affect the runtime of the application [x] header files must be in a -devel package [-] static libraries must be in a -static package [!] containing pkgconfig(.pc) files must 'Requires: pkgconfig' -devel package should have this added. [x] library files that end in .so: go in a -devel package [x] devel pkg: require base package using a fully versioned dependency [x] no .la libtool archives [-] gui app include a %{name}.desktop file [x] must not own files or directories already owned by other packages [x] %install includes rm -rf %{buildroot} [x] filenames in rpm packages must be valid UTF-8 SHOULD [x] test that the package builds in mock http://koji.fedoraproject.org/koji/taskinfo?taskID=1331611 [x] compile and build into binary rpms on all archs http://koji.fedoraproject.org/koji/taskinfo?taskID=1331611 [+] package functions as described [x] those scriptlets are sane [x] subpackages require the base packag fully versioned dep [x] pkgconfig(.pc) in devel [x] no explicit file dep outside /etc, /bin/, /sbin, /usr/{sbin,bin} Add req on pkgconfig in -devel on importing and you are done. libmemcached is APPROVED
Thanks for the review and the head up on license. New Package CVS Request ======================= Package Name: libmemcached Short Description: Client library and command line tools for memcached server Owners: remi Branches: F-10 F-11 InitialCC:
CVS Done
libmemcached-0.28-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/libmemcached-0.28-2.fc11
libmemcached-0.28-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/libmemcached-0.28-2.fc10
libmemcached-0.28-2.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libmemcached'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-4126
libmemcached-0.28-2.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libmemcached'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-4453
libmemcached-0.29-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/libmemcached-0.29-1.fc11
libmemcached-0.29-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/libmemcached-0.29-1.fc10
libmemcached-0.29-1.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libmemcached'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-5236
libmemcached-0.29-1.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libmemcached'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-5268
libmemcached-0.30-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/libmemcached-0.30-1.fc11
libmemcached-0.30-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/libmemcached-0.30-1.fc10
libmemcached-0.30-1.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libmemcached'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-6021
libmemcached-0.30-1.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update libmemcached'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-6437
libmemcached-0.30-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
libmemcached-0.30-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
Will build version 0.31 to have EVR < RHEL-6 Package Change Request ====================== Package Name: libmemcached New Branches: EL-5 Owners: remi InitialCC:
Real branch name Package Change Request ====================== Package Name: libmemcached New Branches: el5 Owners: remi InitialCC:
Git done (by process-git-requests).