[Bug 263181] Review Request: php-pecl-Svn - PHP Subversion Wrapper

[bugzilla at redhat.com]

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 at famillecollet.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fedora at famillecollet.com




------- Additional Comments From fedora at famillecollet.com  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.