Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 168178

Summary: Review request: perl-Class-Autouse - Run-time class loading on first method call
Product: [Fedora] Fedora Reporter: Ralf Corsepius <rc040203>
Component: Package ReviewAssignee: Jose Pedro Oliveira <jose.p.oliveira.oss>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list
Target Milestone: ---Flags: kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://search.cpan.org/dist/Class-Autouse/
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-09-15 18:14:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 163779, 168213    

Description Ralf Corsepius 2005-09-13 09:11:19 UTC
Spec Name or Url:
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-Class-Autouse.spec

SRPM Name or Url:
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-Class-Autouse-1.20-1.src.rpm

Description:
Class::Autouse allows you to specify a class the will only load when a
method of that class is called. For large classes that might not be used
during the running of a program, such as Date::Manip, this can save you
large amounts of memory, and decrease the script load time.

Comment 1 Jose Pedro Oliveira 2005-09-15 01:42:52 UTC
APPROVED

MD5SUMS:
af80dc0e4737a00fead18380c70fa3ca  perl-Class-Autouse-1.20-1.src.rpm

746a4f415dd8c7267bcec07841022275
746a4f415dd8c7267bcec07841022275  Class-Autouse-1.20.tar.gz
c16f62a4fffa4b45795b3eb9308f6ff5  perl-Class-Autouse.spec

Good:
* Package name follows standard
* Tarball MD5 digest verified against CPAN copy
* URL and Source url are valid
* License verified and included
* perl(:MODULE_COMPAT_xxx) present
* perl vendor libs present
* File permissions are valid
* Builds without problems in FC3 and FC4


Potential improvements:

* add the optional build requirement: perl(prefork)
    ...
    t/03_prefork......skipped
        all skipped: prefork.pm is not installed
    t/04_recursive....ok
    ...

    Please add at least this one after importing the package to CVS.

* drop the README file (same content as in the main POD document)

* drop the following superflous build requirments (since perl 5.8.1 at least):

    BuildRequires: »···perl(Test::More) >= 0.47
    BuildRequires: »···perl(File::Spec) >= 0.82
    BuildRequires: »···perl(List::Util) >= 1.11



Comment 2 Jose Pedro Oliveira 2005-09-15 18:14:29 UTC
Closing ticket.  Just pushed into the mirrors.

Comment 3 Ian Burrell 2008-06-13 23:25:00 UTC
Package Change Request
======================
Package Name: perl-Class-Autouse
New Branches: EL-4 EL-5
Updated EPEL Owners: iburrell



Comment 4 Kevin Fenzi 2008-06-16 15:56:26 UTC
cvs done.