Bug 351711 - Review Request: perl-Inline-Files - Allows for multiple inline files in a single perl file
Summary: Review Request: perl-Inline-Files - Allows for multiple inline files in a sin...
Keywords:
Status: CLOSED CURRENTRELEASE
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:
TreeView+ depends on / blocked
 
Reported: 2007-10-25 04:07 UTC by Robin Norwood
Modified: 2008-01-04 05:33 UTC (History)
2 users (show)

Fixed In Version: 0.62-2
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-01-04 05:33:46 UTC
Type: ---
Embargoed:
panemade: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Robin Norwood 2007-10-25 04:07:14 UTC
Spec URL: http://people.redhat.com/rnorwood/rpms/perl-Inline-Files.spec
SRPM URL: http://people.redhat.com/rnorwood/rpms/perl-Inline-Files-0.62-1.fc8.src.rpm
Description: 
Inline::Files generalizes the notion of the `__DATA__' marker and the
associated `<DATA>' filehandle, to an arbitrary number of markers and
associated filehandles.

It is required for some of the tests in the perl-Inline package.
See: https://bugzilla.redhat.com/show_bug.cgi?id=226262

Comment 1 Ralf Corsepius 2007-10-25 04:22:05 UTC
The permissions inside of the tarball are broken. Please change these
permissions in %prep. 

All the chmod's in %install are a side-effect of these broken permissions and
probably can be removed.

Comment 2 Parag AN(पराग) 2007-10-25 09:39:48 UTC
Good to change permissions in %prep

Comment 3 Parag AN(पराग) 2007-11-13 08:56:21 UTC
Robin,
  Can you please provide updates here?

Comment 5 Parag AN(पराग) 2007-11-15 06:31:56 UTC
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
1beb4548be878a28cafb379945c402b8  Inline-Files-0.62.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test outputs
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0,
'blib/lib', 'blib/arch')" t/*.t
t/00init.......ok
t/01basics.....ok
t/02write......ok
t/03nowrite....ok
t/04create.....ok
t/05seek.......ok
t/06backup.....ok
t/07hash.......ok
All tests successful.
Files=8, Tests=67,  3 wallclock secs ( 0.99 cusr +  0.09 csys =  1.08 CPU)
+ Package perl-Inline-Files-0.62-2.fc9 ->
 Provides: perl(Inline::Files) = 0.62 perl(Inline::Files::Data)
perl(Inline::Files::Virtual) = 0.53
 Requires: perl(:MODULE_COMPAT_5.8.8) perl(Carp) perl(Cwd)
perl(Filter::Util::Call) perl(Inline::Files::Virtual) perl(strict) perl(vars)
APPROVED.


Comment 6 Robin Norwood 2007-11-15 22:13:04 UTC
New Package CVS Request
=======================
Package Name: perl-Inline-Files
Short Description: Allows for multiple inline files in a single perl file
Owners: rnorwood
Branches: devel
InitialCC: fedora-perl-devel-list
Cvsextras Commits: yes

Comment 7 Kevin Fenzi 2007-11-16 00:14:39 UTC
cvs done.

Comment 8 Robin Norwood 2007-11-16 20:28:27 UTC
built:http://koji.fedoraproject.org/koji/buildinfo?buildID=24536

Comment 9 Robin Norwood 2007-12-17 20:58:08 UTC
Needed in EPEL for perl-Inline

Package Change Request
======================
Package Name: perl-Inline-Files
New Branches: EL-4 EL-5

Comment 10 Kevin Fenzi 2007-12-17 21:39:45 UTC
cvs done.

Comment 11 Robin Norwood 2008-01-03 15:59:55 UTC
Built for EL-4 and EL-5


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