Bug 439285 - Review Request: php-openid - The PHP OpenID Library
Summary: Review Request: php-openid - The PHP OpenID Library
Status: CLOSED DUPLICATE of bug 227190
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Orion Poplawski
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 439340
TreeView+ depends on / blocked
Reported: 2008-03-27 21:23 UTC by Ian Weller
Modified: 2008-07-05 07:26 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-07-05 07:26:17 UTC
Type: ---
orion: fedora-review-

Attachments (Terms of Use)

Description Ian Weller 2008-03-27 21:23:32 UTC
Request from Infrastructure team.

Spec URL: http://ianweller.fedorapeople.org/SRPMS/php-openid/2.0.1-1/php-openid.spec
SRPM URL: http://ianweller.fedorapeople.org/SRPMS/php-openid/2.0.1-1/php-openid-2.0.1-1.fc8.src.rpm
The PHP OpenID library lets you enable OpenID authentication on sites built
using PHP. It features the OpenID consumer, Store implementations, and an
OpenID server. Features of the 2.x.x series include:
 * Runs on a wide range of PHP versions: from 4.3.0 and up (tested on 5.2.4)
 * Licensed under the Apache Software License.
 * Includes extensive documentation, working example code, and a test suite
 * Attempts to use PHP extensions such as Curl, but does not depend on them
 * Includes extension modules for Attribute Exchange, Simple Registration, and
 * Supports i-names
 * Tested on Windows and Linux
 * Versions 2.x.x support both major OpenID protocol versions transparently

Comment 1 Orion Poplawski 2008-03-28 20:43:19 UTC

- rpmlint checks return nothing
- package meets naming guidelines
- package meets packaging guidelines
- license (ASL 2.0) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
e8be2d48139b4faa45626661c329bdb2  php-openid-2.0.1.tar.bz2
e8be2d48139b4faa45626661c329bdb2  php-openid-2.0.1.tar.bz2.orig
- package compiles on devel (x86)
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 


Why do you need to BR php-common?  It appears to build fine without it.

Comment 2 Ian Weller 2008-03-28 23:02:45 UTC
I realized this after I finished packaging... do I need to reupload the package
to fedorapeople so you can see just that one fix or can I integrate it into CVS
after you approve it? it does build just fine.

Comment 3 Ian Weller 2008-03-29 18:34:21 UTC
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 -

* Sat Mar 29 2008 Ian Weller <ianweller@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...

Comment 4 Ian Weller 2008-03-29 19:10:45 UTC
Giving bug 227190 and bug 227191 some time. Please place review on hold.

Comment 5 Peter Lemenkov 2008-07-05 07:26:17 UTC

*** This bug has been marked as a duplicate of 227190 ***

Note You need to log in before you can comment on or make changes to this bug.