[Bug 694864] Review Request: php-pecl-gearman - PHP wrapper to libgearman

[bugzilla at redhat.com]

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 at ausil.us> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #10 from Dennis Gilmore <dennis at ausil.us> 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.