[Bug 227190] Review Request: php-pear-Auth-OpenID - PHP OpenID

[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-pear-Auth-OpenID - PHP OpenID


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227190


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium
           Priority|normal                      |medium

Fedora at FamilleCollet.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |Fedora at FamilleCollet.com




------- Additional Comments From Fedora at FamilleCollet.com  2007-04-22 03:26 EST -------
I'm quite disappointed by this package...

BuildRequires should be php-pear >= 1:1.4.9-1.2 (older version doesn't provide
the used macros)

PHPUnit version 1.x is obsolete and replace in devel by PHPUnit 3 (which is not
compatible).

Empty build section should be provided (at least to avoid rpmlint warning, i
know there is a bug if there is no %build, even if it only affect arch RPM).

This package is not in default channel (pear.php.net) but in __uri one.
Channels was discussed by packaging commitee but nothing approved.

Uninstall command is :
pear uninstall --nodeps --ignore-errors --register-only %{channel}/%{pear_name}
With %{channel} defined to __uri

And provides should probably be : php-pear(%{channel}/%{pear_name}) = %{version}

Why are you moving %{pear_docdir}/%{pear_name}/doc to docdir and not all
%{pear_docdir}/%{pear_name}/* ?

And version 1.2.2 is available...

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.