[Bug 565232] Review Request: perl-Log-Any - Bringing loggers and listeners together

bugzilla at redhat.com bugzilla at redhat.com
Sat Feb 20 19:37:39 UTC 2010


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Steve Traylen <steve.traylen at cern.ch> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-review+

--- Comment #4 from Steve Traylen <steve.traylen at cern.ch> 2010-02-20 14:37:35 EST ---

Review of perl-Log-Any, February 20th.

yes: rpmlint.
$ rpmlint perl-Log-Any.spec \
   ../RPMS/noarch/perl-Log-Any-0.11-1.fc12.noarch.rpm \
   ../SRPMS/perl-Log-Any-0.11-1.fc12.src.rpm 
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

yes: package name  good for perl.
yes: .spec file name good.
yes: meets pkg guidelines.
yes: spec has gpl+ or Artistic.
yes: yes source is same as perl so as above.
yes: english .spec file.
yes: .spec is legible.
yes: matches upstream.
$ md5sum Log-Any-0.11.tar.gz SOURCES/Log-Any-0.11.tar.gz 
9186d94ee2ae7cada408fc5faf86d975  Log-Any-0.11.tar.gz
9186d94ee2ae7cada408fc5faf86d975  SOURCES/Log-Any-0.11.tar.gz

yes: compiles fine.
yes: buildrequires sane.
yes: no locale.
yes: no shared libs.
yes: no system libs.
yes: not relocatable.
yes: Owns all directories.
yes: no duplicates.
yes: %clean actually cleans.
yes: No large docs.
yes: No -devel package.
yes: no pkgconfig
yes: no sub-packages.
yes: no gui.
yes: does not own directories from other packages.
yes: %install rm -rf first.
yes: UTF8

Builds in mock-rawhide (F14!)

Basically a very simple cpanspec generated package.

APPROVED.

 Steve

p.s.

(I've got a perl module perl-VOMS-Lite, up for review if you have time
 to review.)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.



More information about the package-review mailing list