Bug 866370

Summary: Review Request: php-channel-theseer - Adds theseer channel to PEAR
Product: [Fedora] Fedora Reporter: Remi Collet <fedora>
Component: Package ReviewAssignee: Johan Cwiklinski <fedora>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora, notting, package-review
Target Milestone: ---Flags: fedora: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-20 15:22:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 866428    
Attachments:
Description Flags
fedora review output none

Description Remi Collet 2012-10-15 08:38:04 UTC
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

Comment 1 Johan Cwiklinski 2012-10-20 16:58:01 UTC
Created attachment 630488 [details]
fedora review output

Comment 2 Johan Cwiklinski 2012-10-20 17:04:37 UTC
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.

Comment 3 Remi Collet 2012-10-20 17:14:25 UTC
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:

Comment 4 Gwyn Ciesla 2012-10-22 12:06:18 UTC
Git done (by process-git-requests).

Comment 5 Fedora Update System 2012-10-22 12:57:12 UTC
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

Comment 6 Fedora Update System 2012-10-22 13:00:14 UTC
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

Comment 7 Fedora Update System 2012-10-22 13:00:43 UTC
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

Comment 8 Fedora Update System 2012-10-22 20:56:08 UTC
php-channel-theseer-1.3-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 9 Fedora Update System 2012-12-20 15:22:57 UTC
php-channel-theseer-1.3-1.el6 has been pushed to the Fedora EPEL 6 stable repository.