https://bugzilla.redhat.com/show_bug.cgi?id=866370
Bug ID: 866370 QA Contact: extras-qa@fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting@redhat.com, package-review@lists.fedoraproject.org Assignee: nobody@fedoraproject.org Summary: Review Request: php-channel-theseer - Adds theseer channel to PEAR Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: fedora@famillecollet.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora
Spec URL: http://remi.fedorapeople.org/php-channel-theseer.spec SRPM URL: http://remi.fedorapeople.org/php-channel-theseer-1.3-1.fc18.src.rpm Description: This package adds the pear.netpirates.net (theseer) channel which allows PEAR packages from this channel to be installed.
Fedora Account System Username: remi
https://bugzilla.redhat.com/show_bug.cgi?id=866370
Remi Collet fedora@famillecollet.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |866428
https://bugzilla.redhat.com/show_bug.cgi?id=866370
Johan Cwiklinski johan@x-tnd.be changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |johan@x-tnd.be Assignee|nobody@fedoraproject.org |johan@x-tnd.be Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #1 from Johan Cwiklinski johan@x-tnd.be --- Created attachment 630488 --> https://bugzilla.redhat.com/attachment.cgi?id=630488&action=edit fedora review output
https://bugzilla.redhat.com/show_bug.cgi?id=866370
Johan Cwiklinski johan@x-tnd.be changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+
--- Comment #2 from Johan Cwiklinski johan@x-tnd.be --- Package builds fine in mock.
rpmlint output: php-channel-theseer.noarch: W: spelling-error %description -l en_US netpirates -> net pirates, net-pirates, penetrates php-channel-theseer.noarch: W: no-documentation php-channel-theseer.src: W: spelling-error %description -l en_US netpirates -> net pirates, net-pirates, penetrates php-channel-theseer.src:24: W: unversioned-explicit-provides php-channel(%{channel})
All of those warnings can be ignored.
Require issues pointed in fedora-review output is false positive, the licencing issue can be ignored, as weel as the missing %{name} prefix in Source0.
The package respects php pear channel packaging guidelines.
APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=866370
Remi Collet fedora@famillecollet.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs?
--- Comment #3 from Remi Collet fedora@famillecollet.com --- Thanks for the review.
New Package SCM Request ======================= Package Name: php-channel-theseer Short Description: Adds theseer channel to PEAR Owners: remi Branches: f17 f18 el6 InitialCC:
https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #4 from Jon Ciesla limburgher@gmail.com --- Git done (by process-git-requests).
https://bugzilla.redhat.com/show_bug.cgi?id=866370
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |MODIFIED
https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #5 from Fedora Update System updates@fedoraproject.org --- php-channel-theseer-1.3-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/php-channel-theseer-1.3-1.el6
https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #6 from Fedora Update System updates@fedoraproject.org --- php-channel-theseer-1.3-1.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/php-channel-theseer-1.3-1.fc18
https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #7 from Fedora Update System updates@fedoraproject.org --- php-channel-theseer-1.3-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/php-channel-theseer-1.3-1.fc17
https://bugzilla.redhat.com/show_bug.cgi?id=866370
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #8 from Fedora Update System updates@fedoraproject.org --- php-channel-theseer-1.3-1.el6 has been pushed to the Fedora EPEL 6 testing repository.
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866370
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed| |2012-12-20 10:22:54
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=866370
--- Comment #9 from Fedora Update System updates@fedoraproject.org --- php-channel-theseer-1.3-1.el6 has been pushed to the Fedora EPEL 6 stable repository.
package-review@lists.fedoraproject.org