Bug 461303

Fails to build for me (in rawhide); the tests fail due to a lack of Test::More.  Even by-the-book cpanspec-generated packages may be missing a few things.  I added a build dependency on perl(Test::More) and things build OK, so I'll just assume that it's there and go on.

Otherwise everything looks fine.

* source files match upstream:
   57e73cc7ef596d9b65288223daa252fe1e2cb9a07b79f7e8786c86c6b7e4bbb9  
   Data-Section-0.005.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.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper (after adding perl(Test::More)).
* %clean is present.
* package builds in mock (rawhide, x86_64) (after fixing build deps).
* package installs properly.
* rpmlint is silent.
* final provides and requires are sane:
   perl(Data::Section) = 0.005
   perl-Data-Section = 0.005-1.fc10
  =
   perl(:MODULE_COMPAT_5.10.0)
   perl(Class::ISA)
   perl(Sub::Exporter) >= 0.979
   perl(strict)
   perl(warnings)

* %check is present and all tests pass:
   All tests successful.
   Files=3, Tests=19,  0 wallclock secs ( 0.02 usr  0.01 sys +  0.07 cusr  0.01 
    csys =  0.11 CPU)

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

APPROVED, just fix that build dep so that you can actually build the package.

Thanks for the review - I always forget the Test::More BR as its always present on my installs. Will do a proper mock build next time to validate it.

Can you add a cvs request here? 
See: http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

New Package CVS Request
=======================
Package Name: perl-Data-Section
Short Description: Read multiple hunks of data out of your DATA section
Owners: berrange
Branches: F-9
InitialCC:

Daniel, one piece of recommendation:

Would you please consider to add "InitialCC: perl-sig" to your perl packages? Many perl packagers apply this, because this causes BZ's and commits to be CC:'ed to the fedora-perl-devel list, thus automatically notifies them about issues with perl packages - Thanks in advance.

cvs done.

Ralf, looks like PkgDB is already setup, so too late to add InitialCC - someone will have to request it in pkgdb directly. I'll add it to any future perl RPMs I add

Build now in rawhide:

$ koji latest-pkg dist-f10 perl-Data-Section
Build                                     Tag                   Built by
----------------------------------------  --------------------  ----------------
perl-Data-Section-0.005-2.fc10            dist-f10              berrange

Package Change Request
======================
Package Name: perl-Data-Section
New Branches: EL-6
Owners: berrange

CVS done (by process-cvs-requests.py).

Hi Daniel,

can we have an EL-5 branch for this package please? I have already requested the necessary update of Sub::Exporter (Bug #799993) and have a patch to get the test suite to work with Test::More < 0.88 as found in EL-5. I (pghmcfc) would be happy to maintain this branch if you're not interested.

Package Change Request
======================
Package Name: perl-Data-Section
New Branches: EL-5
Owners: berrange

Created attachment 567930 [details]
Patch to get test suite to work with Test::More < 0.88

Git done (by process-git-requests).

perl-Data-Section-0.101621-2.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/perl-Data-Section-0.101621-2.el5

perl-Data-Section-0.101621-2.el5 has been pushed to the Fedora EPEL 5 stable repository.