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 <fedora@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fedora@xxxxxxxxxxxxxxxxx --- Comment #1 from Remi Collet <fedora@xxxxxxxxxxxxxxxxx> 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 Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review