Bug 922745 - Review Request: perl-Module-Load - Run-time require of both modules and files
Summary: Review Request: perl-Module-Load - Run-time require of both modules and files
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Petr Šabata
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-03-18 12:18 UTC by Petr Pisar
Modified: 2013-03-25 14:21 UTC (History)
3 users (show)

Fixed In Version: perl-Module-Load-0.24-1.fc20
Clone Of:
Environment:
Last Closed: 2013-03-25 14:21:57 UTC
Type: ---
Embargoed:
psabata: fedora-review+


Attachments (Terms of Use)

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.


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