[Bug 497688] Review Request: php-pecl-memcached - Extension to work with the Memcached caching daemon

bugzilla at redhat.com bugzilla at redhat.com
Sat May 2 18:17:49 UTC 2009


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=497688


Guillaume Kulakowski <llaumgui at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |llaumgui at gmail.com




--- Comment #1 from Guillaume Kulakowski <llaumgui at gmail.com>  2009-05-02 14:17:48 EDT ---
+ rpmlint is ok
    php-pecl-memcached.x86_64: I: checking
    php-pecl-memcached-debuginfo.x86_64: I: checking
    php-pecl-memcached.src: I: checking
    3 packages and 1 specfiles checked; 0 errors, 0 warnings.
+ package name ok
+ spec file name ok
+ package meet the PHP Guidelines
+ License ok : PHP
+ License is upstream 
+ spec in english and legible
+ license file in sources is provided
+ sources match the upstream sources
646c3ffb622adc0379a3951155f7f4bb  memcached-0.1.5.tgz
+ Source URL ok
+ build  on F10.x86_64
BuildRequires is ok
    php-devel >= 5.2.0, php-pear
    libmemcached-devel, zlib-devel
+ no locale
+ no .so


+ own all directories that it creates
+ no duplicate file
+ %defattr ok
+ %clean section
+ use macros consistently
+ contain code
+ small documentation not required to run
+ devel package OK
+ no pkgconfig
+ no sub-package
+ no GUI
+ don't own files or directories already owned by other packages
+ %install start with rm -rf 
+ valid UTF-8
+ build in mock (F-10 i386 & x86_64)
+ no test suite
+ scriptlets ok
+ Final Requires ok
    php(zend-abi) = %{php_zend_api}
    php(api) = %{php_core_api}
+ Final Provides ok
    php-pecl(%{pecl_name}) = %{version}-%{release}

--------
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.




More information about the Fedora-package-review mailing list