Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=694864 Dennis Gilmore <dennis@xxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #10 from Dennis Gilmore <dennis@xxxxxxxx> 2011-04-11 16:56:59 EDT --- looks cleaner now sources match upstream sha512sum gearman-0.7.0.tgz fedora/SOURCES/gearman-0.7.0.tgz 0ba0779cf5b22744488b8c78d5a2479a9910a9f5f8a192c6487e1109bef2a8ab4a8821917b9f04ee0c2df9ad54c15bf3b549f970c9c62fb579a9cd15e1125fd4 gearman-0.7.0.tgz 0ba0779cf5b22744488b8c78d5a2479a9910a9f5f8a192c6487e1109bef2a8ab4a8821917b9f04ee0c2df9ad54c15bf3b549f970c9c62fb579a9cd15e1125fd4 fedora/SOURCES/gearman-0.7.0.tgz rpmlint /var/lib/mock/fedora-15-x86_64/result php-pecl-gearman.x86_64: W: private-shared-object-provides /usr/lib64/php/modules/gearman.so gearman.so()(64bit) php-pecl-gearman.src: W: spelling-error Summary(en_US) libgearman -> Lieberman php-pecl-gearman.src: W: spelling-error %description -l en_US libgearman -> Lieberman 3 packages and 0 specfiles checked; 0 errors, 3 warnings. of those the only one we could look at removing is the provides on gearman.so()(64bit) only tweak i can see would be nice is to use the %{version} macro in the sourceurl but that is minor and not worth nitpicking. approved -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review