Bug 979653 - Review Request: perl-Perl6-Slurp - Implemention of the Perl 6 'slurp' built-in
Summary: Review Request: perl-Perl6-Slurp - Implemention of the Perl 6 'slurp' built-in
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 987717
TreeView+ depends on / blocked
 
Reported: 2013-06-29 05:50 UTC by Christopher Meng
Modified: 2013-07-24 01:29 UTC (History)
3 users (show)

Fixed In Version: perl-Perl6-Slurp-0.051003-2.fc18
Clone Of:
Environment:
Last Closed: 2013-07-15 01:10:19 UTC
Type: ---
Embargoed:
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Christopher Meng 2013-06-29 05:50:28 UTC
Spec URL: http://cicku.me/perl-Perl6-Slurp.spec
SRPM URL: http://cicku.me/perl-Perl6-Slurp-0.051003-1.fc20.src.rpm
Description: slurp takes:

a filename,
a filehandle,
a typeglob reference,
an IO::File object, or
a scalar reference,

converts it to an input stream (using open() if necessary), 
and reads in the entire stream. If slurp fails to set up or 
read the stream, it throws an exception.
Fedora Account System Username: cicku

Comment 1 Parag AN(पराग) 2013-07-03 03:41:58 UTC
Review:

+ mock build is successful for f20

+ rpmlint on rpms gave
perl-Perl6-Slurp.noarch: W: spelling-error Summary(en_US) Implemention -> Implement ion, Implement-ion, Implementation
perl-Perl6-Slurp.noarch: W: spelling-error %description -l en_US filename -> file name, file-name, filament
perl-Perl6-Slurp.noarch: W: spelling-error %description -l en_US filehandle -> file handle, file-handle, mishandle
perl-Perl6-Slurp.noarch: W: spelling-error %description -l en_US typeglob -> type glob, type-glob, logotype
perl-Perl6-Slurp.src: W: spelling-error Summary(en_US) Implemention -> Implement ion, Implement-ion, Implementation
perl-Perl6-Slurp.src: W: spelling-error %description -l en_US filename -> file name, file-name, filament
perl-Perl6-Slurp.src: W: spelling-error %description -l en_US filehandle -> file handle, file-handle, mishandle
perl-Perl6-Slurp.src: W: spelling-error %description -l en_US typeglob -> type glob, type-glob, logotype
2 packages and 0 specfiles checked; 0 errors, 8 warnings.
==> Ignore this

+ Source verified with upstream as (sha256sum)
srpm tarball: 49fdcbb37b7a0fd1bf043970cfe42913080b1e3bf794a03b103c14480d48d60a
upstream tarball: 49fdcbb37b7a0fd1bf043970cfe42913080b1e3bf794a03b103c14480d48d60a

- License "Freely redistributable without restriction" looks invalid.

+ rest follows packaging guidelines

+ make test is successful with output
All tests successful.
Files=13, Tests=183,  1 wallclock secs ( 0.06 usr  0.01 sys +  0.26 cusr  0.03 csys =  0.36 CPU)

+ This package
Provides: perl(Perl6::Slurp) = 0.051003 perl-Perl6-Slurp = 0.051003-1.fc20
Requires: perl >= 0:5.008 perl(Carp) perl(Scalar::Util) perl(strict) perl(warnings)

suggestions:
1)Look https://fedoraproject.org/wiki/Packaging:Perl#License_tag and you will see license for this package is "GPL+ or Artistic"

Comment 2 Christopher Meng 2013-07-03 05:39:43 UTC
Hi,

I want to know if I need to add Requires of perl(strict) 

As a review did by ppisar(https://bugzilla.redhat.com/show_bug.cgi?id=979648), he recommended me to add this. Although I don't think it's a problem, because strict is in perl main package now. 

NEW SPEC URL: http://cicku.me/perl-Perl6-Slurp.spec
NEW SRPM URL: http://cicku.me/perl-Perl6-Slurp-0.051003-2.fc20.src.rpm

Comment 3 Parag AN(पराग) 2013-07-03 06:34:55 UTC
I don't think you need to add perl(strict) requires.

APPROVED above srpm.

Comment 4 Christopher Meng 2013-07-03 09:43:26 UTC
New Package SCM Request
=======================
Package Name: perl-Perl6-Slurp
Short Description: Implemention of the Perl 6 'slurp' built-in
Owners: cicku
Branches: f18 f19
InitialCC:

Comment 5 Gwyn Ciesla 2013-07-03 12:55:33 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2013-07-05 15:32:20 UTC
perl-Perl6-Slurp-0.051003-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/perl-Perl6-Slurp-0.051003-2.fc19

Comment 7 Fedora Update System 2013-07-05 15:33:00 UTC
perl-Perl6-Slurp-0.051003-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/perl-Perl6-Slurp-0.051003-2.fc18

Comment 8 Fedora Update System 2013-07-06 00:50:06 UTC
perl-Perl6-Slurp-0.051003-2.fc18 has been pushed to the Fedora 18 testing repository.

Comment 9 Fedora Update System 2013-07-15 01:10:19 UTC
perl-Perl6-Slurp-0.051003-2.fc19 has been pushed to the Fedora 19 stable repository.

Comment 10 Fedora Update System 2013-07-15 01:10:30 UTC
perl-Perl6-Slurp-0.051003-2.fc18 has been pushed to the Fedora 18 stable repository.


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