[Bug 439285] Review Request: php-openid - The PHP OpenID Library

bugzilla at redhat.com bugzilla at redhat.com
Sat Mar 29 18:34:21 UTC 2008


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-openid - The PHP OpenID Library


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


ianweller at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEEDINFO
               Flag|                            |needinfo?(orion at cora.nwra.co
                   |                            |m)




------- Additional Comments From ianweller at gmail.com  2008-03-29 14:34 EST -------
there were more issues that i found myself... version 2.0.1-2.
spec - http://ianweller.fedorapeople.org/SRPMS/php-openid/2.0.1-2/php-openid.spec
srpm -
http://ianweller.fedorapeople.org/SRPMS/php-openid/2.0.1-2/php-openid-2.0.1-2.fc8.src.rpm

* Sat Mar 29 2008 Ian Weller <ianweller at gmail.com> 2.0.1-2
- Removed unnecessary BuildRequires
- Added provides php-yadis, since it does
- Put everything under an Auth directory in datadir/php, because otherwise the
code wouldn't work

Also, another issue that has come up: if this package were to pass review, it
would supersede bug 227190 and bug 227191, two packages for OpenID and Yadis in
PHP. I've contacted the packager about this, and I've looked at the review
requests, and it seems like they shouldn't be provided under the php-pear name,
since I can't find OpenID in pear, and Yadis is out of date compared to URL0 for
this package.

orion: are you still willing to review this? i noticed you set fedora-review to
-. i don't usually see that, i usually see them leave it at ? when there's still
fixable issues...

-- 
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.




More information about the Fedora-package-review mailing list