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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]




------- Additional Comments From [EMAIL PROTECTED]  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
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to