Bug 922745

Summary: Review Request: perl-Module-Load - Run-time require of both modules and files
Product: [Fedora] Fedora Reporter: Petr Pisar <ppisar>
Component: Package ReviewAssignee: Petr Šabata <psabata>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: notting, package-review, psabata
Target Milestone: ---Flags: psabata: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: perl-Module-Load-0.24-1.fc20 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-03-25 14:21:57 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:

Description Petr Pisar 2013-03-18 12:18:58 UTC
Spec URL: http://ppisar.fedorapeople.org/perl-Module-Load/perl-Module-Load.spec
SRPM URL: http://ppisar.fedorapeople.org/perl-Module-Load/perl-Module-Load-0.24-1.fc20.src.rpm
Description:
If you consult "perldoc -f require" you will see that "require" will behave
differently when given a bare-word or a string. In the case of a string,
"require" assumes you are wanting to load a file. But in the case of
a bare-word, it assumes you mean a module.

This gives nasty overhead when you are trying to dynamically require modules
at run-time, since you will need to change the module notation to a file
notation fitting the particular platform you are on.

"load" eliminates the need for this overhead and will just DWYM.


Fedora Account System Username: ppisar

Comment 1 Petr Pisar 2013-03-18 12:19:56 UTC
This package will dual-live with perl core modules.

Comment 2 Petr Šabata 2013-03-19 15:16:25 UTC
No issues whatsoever, approving.

Comment 3 Petr Pisar 2013-03-20 07:47:40 UTC
New Package SCM Request
=======================
Package Name: perl-Module-Load
Short Description: Run-time require of both modules and files
Owners: ppisar jplesnik psabata
Branches: 
InitialCC: perl-sig

Comment 4 Gwyn Ciesla 2013-03-20 12:51:14 UTC
Unretired.

Comment 5 Petr Pisar 2013-03-20 14:02:34 UTC
Waiting on unblocking <https://fedorahosted.org/rel-eng/ticket/5543>.

Comment 6 Petr Pisar 2013-03-25 14:21:57 UTC
Thank you for the review and the repository.