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

bugzilla at redhat.com bugzilla at redhat.com
Sun Apr 22 08:30:14 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





------- Additional Comments From Axel.Thimm at ATrpms.net  2007-04-22 04:30 EST -------
(In reply to comment #1)
> I'm quite disappointed by this package...

I'm very sorry about that, but I followed the packaging guidelines which provide
explicit BuildRequires/Requires for pear packages.

http://fedoraproject.org/wiki/Packaging/PHP?action=recall&rev=2

If you consider the guidelines wrong, then please raise that as a general issue
on the packaging list, ranting on a specific package won't help.

> Channels was discussed by packaging commitee but nothing approved.

If nothing was approved/rejected, this means that the current guidelines still
apply, and you should try to get the FPC to modify them if need be.

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

Other than rpmlint not liking it there is no "bug".

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

That's bad. So we need a PHPUnit 1.x compat :/

> And version 1.2.2 is available...

Not when I submitted the package ;)

Does 1.2.2 work with PHPUnit 3?

-- 
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 Fedora-package-review mailing list