Bug 663213 - Review Request: perl-File-Inplace - Perl module for in-place editing of files
Summary: Review Request: perl-File-Inplace - Perl module for in-place editing of files
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Iain Arnell
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 664359
TreeView+ depends on / blocked
 
Reported: 2010-12-15 00:25 UTC by Ruediger Landmann
Modified: 2010-12-22 01:35 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-12-22 01:35:18 UTC
Type: ---
Embargoed:
iarnell: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Ruediger Landmann 2010-12-15 00:25:54 UTC
Spec URL: 
http://rlandmann.fedorapeople.org/perl-File-Inplace/perl-File-Inplace.spec

SRPM URL:
http://rlandmann.fedorapeople.org/perl-File-Inplace/perl-File-Inplace-0.20-1.fc14.src.rpm

Description: 
File::Inplace is a perl module intended to ease the common task of editing
a file in-place. Inspired by variations of perl's -i option, this module is
intended for somewhat more structured and reusable editing than command
line perl typically allows. File::Inplace endeavors to guarantee file
integrity; that is, either all of the changes made will be saved to the
file, or none will. It also offers functionality such as backup creation,
automatic field splitting per-line, automatic chomping/unchomping, and
aborting edits partially through without affecting the original file.

rpmlint output:
$ rpmlint SPECS/perl-File-Inplace.spec 
SPECS/perl-File-Inplace.spec: W: no-cleaning-of-buildroot %clean
SPECS/perl-File-Inplace.spec: W: no-buildroot-tag
SPECS/perl-File-Inplace.spec: W: no-%clean-section
0 packages and 1 specfiles checked; 0 errors, 3 warnings.

$ rpmlint SRPMS/perl-File-Inplace-0.20-1.fc14.src.rpm
perl-File-Inplace.src: W: spelling-error %description -l en_US unchomping -> uncomplaining, uncompromising, uncomprehending
perl-File-Inplace.src: W: no-cleaning-of-buildroot %clean
perl-File-Inplace.src: W: no-buildroot-tag
perl-File-Inplace.src: W: no-%clean-section
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

$ rpmlint RPMS/noarch/perl-File-Inplace-0.20-1.fc14.noarch.rpm
perl-File-Inplace.noarch: W: spelling-error %description -l en_US unchomping -> uncomplaining, uncompromising, uncomprehending
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Scratchbuilds in Koji:
F15 -- https://koji.fedoraproject.org/koji/taskinfo?taskID=2664417
F14 -- https://koji.fedoraproject.org/koji/taskinfo?taskID=2664684

Comment 1 Iain Arnell 2010-12-17 16:06:40 UTC
How did I never spot this module before?

Comment 2 Iain Arnell 2010-12-17 16:14:05 UTC
+ source files match upstream.  
    fae0848ebef6969502dde017805a31a1  File-Inplace-0.20.tar.gz

+ package meets naming and versioning guidelines.
+ specfile is properly named, is cleanly written and uses macros consistently.
+ summary is OK.
+ description is OK.
+ dist tag is present.
+ license field matches the actual license.
    GPL+ or Artistic

+ license is open source-compatible.
+ license text not included upstream.
+ latest version is being packaged.
+ BuildRequires are proper.
+ compiler flags are appropriate.
+ package builds in mock
    https://koji.fedoraproject.org/koji/taskinfo?taskID=2664417

+ package installs properly.
+ rpmlint has no useful complaints:
    2 packages and 0 specfiles checked; 0 errors, 5 warnings.
    [iarnell@microstar perl-File-Inplace]$ rpmlint -v ~/dl/perl-File-Inplace-0.20-1.fc15.*rpm
    perl-File-Inplace.noarch: I: checking
    perl-File-Inplace.noarch: W: spelling-error %description -l en_US unchomping -> uncomplaining, uncompromising, uncomprehending
    perl-File-Inplace.noarch: I: checking-url http://search.cpan.org/dist/File-Inplace/ (timeout 10 seconds)
    perl-File-Inplace.src: I: checking
    perl-File-Inplace.src: W: spelling-error %description -l en_US unchomping -> uncomplaining, uncompromising, uncomprehending
    perl-File-Inplace.src: I: checking-url http://search.cpan.org/dist/File-Inplace/ (timeout 10 seconds)
    perl-File-Inplace.src: W: no-cleaning-of-buildroot %clean
    perl-File-Inplace.src: W: no-buildroot-tag
    perl-File-Inplace.src: W: no-%clean-section
    perl-File-Inplace.src: I: checking-url http://www.cpan.org/modules/by-module/File/File-Inplace-0.20.tar.gz (timeout 10 seconds)
    2 packages and 0 specfiles checked; 0 errors, 5 warnings.

+ final provides and requires are sane:
    perl(File::Inplace) = 0.20
    perl-File-Inplace = 0.20-1.fc15
=
    perl(Carp)  
    perl(File::Basename)  
    perl(File::Copy)  
    perl(File::Temp)  
    perl(IO::File)  
    perl(IO::Handle)  
    perl(:MODULE_COMPAT_5.12.2)  
    perl(strict)  
    rpmlib(CompressedFileNames) <= 3.0.4-1
    rpmlib(FileDigests) <= 4.6.0-1
    rpmlib(PayloadFilesHavePrefix) <= 4.0-1
    rpmlib(PayloadIsXz) <= 5.2-1
    rpmlib(VersionedDependencies) <= 3.0.3-1
+ %check is present and all tests pass.
    + make test
    PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
    t/File-Inplace.t .. ok
    All tests successful.
    Files=1, Tests=21,  0 wallclock secs ( 0.03 usr  0.01 sys +  0.07 cusr  0.00 csys =  0.11 CPU)
    Result: PASS

+ no shared libraries are added to the regular linker search paths.
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no generically named files
+ code, not content.
+ documentation is small, so no -doc subpackage is necessary.
+ %docs are not necessary for the proper functioning of the package.


APPROVED.

Comment 3 Ruediger Landmann 2010-12-20 03:59:45 UTC
Thanks for the review, Iain :)

New Package SCM Request
=======================
Package Name: perl-File-Inplace
Short Description: Perl module for in-place editing of files
Owners: rlandmann
Branches: f13 f14 el6
InitialCC: perl-sig

Comment 4 Ruediger Landmann 2010-12-20 04:00:33 UTC
Sorry -- should do EL5 too:

New Package SCM Request
=======================
Package Name: perl-File-Inplace
Short Description: Perl module for in-place editing of files
Owners: rlandmann
Branches: f13 f14 el5 el6
InitialCC: perl-sig

Comment 5 Kevin Fenzi 2010-12-21 06:20:16 UTC
Git done (by process-git-requests).


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