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


Remi Collet <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]




--- Comment #1 from Remi Collet <[email protected]>  2008-12-28 11:46:21 
EDT ---
A few notes :

- License is PHP, not BSD (according to pecl.php.net)
- must use %setup -q -c (to not have package.xml outside the build tree)
- missing require for ABI check : php(zend-abi)
- should use %{pecl_install} and  %{pecl_uninstall} when exists
- why PEAR in sumnary ?
- should add example directory in %doc (rather than each files)

Read : http://fedoraproject.org/wiki/Packaging/PHP

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

_______________________________________________
Fedora-package-review mailing list
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to