[Bug 866370] Review Request: php-channel-theseer - Adds theseer channel to PEAR

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=866370

Johan Cwiklinski <johan@xxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |
              Flags|                            |fedora-review+

--- Comment #2 from Johan Cwiklinski <johan@xxxxxxxx> ---
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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



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