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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |chris.stone@xxxxxxxxx




------- Additional Comments From chris.stone@xxxxxxxxx  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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]