Bug 589479 - Review Request: perl-Compress-Raw-Lzma - Low-level interface to lzma compression library
Summary: Review Request: perl-Compress-Raw-Lzma - Low-level interface to lzma compress...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jochen Schmitt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 589482
TreeView+ depends on / blocked
 
Reported: 2010-05-06 09:32 UTC by Paul Howarth
Modified: 2010-05-12 14:33 UTC (History)
3 users (show)

Fixed In Version: perl-Compress-Raw-Lzma-2.029-2.fc13
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-05-12 12:22:33 UTC
Type: ---
Embargoed:
jochen: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Paul Howarth 2010-05-06 09:32:17 UTC
Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Compress-Raw-Lzma/branches/fedora/perl-Compress-Raw-Lzma.spec
SRPM URL: http://www.city-fan.org/~paul/extras/perl-Compress-Raw-Lzma/perl-Compress-Raw-Lzma-2.028-1.fc14.src.rpm
Description:

This module provides a Perl interface to the lzma compression library.
It is used by IO::Compress::Lzma.

Comment 1 Jochen Schmitt 2010-05-06 17:51:18 UTC
GOOD:
+ Package name fullfill naming guidelines.
+ URL tag contains proper project home page.
+ Package contains most recent release of the software.
+ Could download source file via spectool -g
+ Consistantly rpm macro usages.
+ License tag contains proper OSS licenses
+ Package has proper BuildRoot definition.
+ Package contains versioned MODULE_cOMPAT_ Req.
+ Package contains no subpackages
+ Build use RPM_OPT_FLAGS
+ Build use SMP flags
+ Local build works fine.
+ Mockbuild works fine.
+ Build on koji works fine.
+ Packaged sources matches with upstream.
* Rpmlint has no complains about the binary package
+ Rpmlint has no complains about the debuginfo package
+ Debuginfo package contains sources
+ %files stanza use rpm macros
+ %files has proper attributes
+ All files of the package belong to the package
+ Package contains no files belog to an other package
+ %doc stanza is small, so we ne no extra doc subpackage
+ Package has proper %ChangeLog

BAD:
- Package contains no verbatin copy of the license text.
- Package has no %clean section, so it's only valid for F-13 and devel
- Rpmlint shows warnings for the source RPM_OPT_FLAGS$ rpmlint perl-Compress-Raw-Lzma-2.028-1.fc12.src.rpm 
perl-Compress-Raw-Lzma.src: W: no-cleaning-of-buildroot %clean
perl-Compress-Raw-Lzma.src: W: no-%clean-section
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

-

Comment 2 Paul Howarth 2010-05-06 18:48:01 UTC
- Package contains no verbatim copy of the license text.

The upstream tarball doesn't contain one, as usual for perl modules. The license terms are clearly stated as "same as perl" in the code itself though.

- Package has no %clean section, so it's only valid for F-13 and devel
- Rpmlint shows warnings for the source RPM_OPT_FLAGS$ rpmlint
perl-Compress-Raw-Lzma-2.028-1.fc12.src.rpm 
perl-Compress-Raw-Lzma.src: W: no-cleaning-of-buildroot %clean
perl-Compress-Raw-Lzma.src: W: no-%clean-section
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

These are intended. The only current user of Compress::Raw::Lzma is IO::Compress::Lzma (see Bug #589482) and that requires IO::Compress::Base >= 2.027, which only went into Rawhide today. I don't expect to be building this for F-12 or earlier releases any time soon, especially since IO::Compress::Base is a core module and unlikely to be updated there unless there's a real bug.

Comment 4 Jochen Schmitt 2010-05-09 18:19:17 UTC
Bood:
+ Could download upstream sources via spectool -g
+ Packaged sources matches with updateam
(md5sum: 96ed2d92d64c911e4c5e0b6cdd34f13b)
+ Local build works fine
+ Build honours RPM_OPT_FLAGS
+ Build is SMP compilant
+ Scratch build on koji works fine

*** APPROVED for devel and F-13 ***

Comment 5 Paul Howarth 2010-05-09 20:24:56 UTC
New Package CVS Request
=======================
Package Name: perl-Compress-Raw-Lzma
Short Description: Low-level interface to lzma compression library
Owners: pghmcfc
Branches: F-13
InitialCC: perl-sig

Comment 6 Kevin Fenzi 2010-05-11 04:42:31 UTC
CVS done (by process-cvs-requests.py).

Remember to assign the review to the package reviewer. ;)

Comment 7 Fedora Update System 2010-05-11 10:02:58 UTC
perl-Compress-Raw-Lzma-2.029-2.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/perl-Compress-Raw-Lzma-2.029-2.fc13

Comment 8 Fedora Update System 2010-05-12 12:22:28 UTC
perl-Compress-Raw-Lzma-2.029-2.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Paul Howarth 2010-05-12 14:33:35 UTC
All done; thanks for the review Jochen.


Note You need to log in before you can comment on or make changes to this bug.