Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pecl-Svn - PHP Subversion Wrapper https://bugzilla.redhat.com/show_bug.cgi?id=263181 fedora@xxxxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fedora@xxxxxxxxxxxxxxxxx ------- Additional Comments From fedora@xxxxxxxxxxxxxxxxx 2007-09-09 02:15 EST ------- First look at this review : Have a look to lastest PECL Guidelines (approved but not yet inline) : http://fedoraproject.org/wiki/PackagingDrafts/PHP You should note : - API check : php(zend-abi) - Register the extension You could have a look, in rawhide, to php-pecl-phar or php-pecl-memcache recent package. AS package name is svn, you must use php-pecl-svn (lowercase) No need to "strip" library, this is done by rpmbuild (need for -debuginfo subpackage) No Need for the License file (Guidelines says If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc) You should also note that : - this is a "beta" version - no update since 2006/03 - 13 open bugs -- 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, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review