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

bugzilla at redhat.com bugzilla at redhat.com
Sun Apr 22 19:05:07 UTC 2007


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


chris.stone at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |chris.stone at gmail.com




------- Additional Comments From chris.stone at gmail.com  2007-04-22 15:05 EST -------
The guidelines need updating, thanks Axel for pointing this out.  We definately
need to get channel stuff in the guidelines, I did not think it important before
because I didn't think we'd have any packages that used other channels.  It
appears I was wrong.

I think any attempt at review on this package should probably wait until the
pear version # and channel guidelines are in approved by the packaging comittee.

In the meantime, Axel, can you please investigate the possibilities of modifying
this to work with PHPUnit3?  It may be that we will need a PHPUnit1-compat rpm
or something, so we should see how difficult it is to either package a compat
rpm or modify this rpm to work with PHPUnit3.

All the other stuff seems like politics to me, I would recommend that you just
use the php-pear template.  It appears you did because of some comments I see in
the spec, but I wouldn't bother removing stuff like %build or changing
%buildroot.  Thats just nonsense stuff and I'd rather focus on the more
important issues.

Thanks.

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




More information about the package-review mailing list