Bug 205060 (perl-Sub-Name)

Summary: Review Request: perl-Sub-Name - Name -- or rename -- a sub
Product: [Fedora] Fedora Reporter: Chris Weyl <cweyl>
Component: Package ReviewAssignee: Patrice Dumas <pertusus>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideFlags: wtogami: fedora-cvs+
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://search.cpan.org/dist/Sub-Name/
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-09-06 14:57:53 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, 205318, 205321    

Description Chris Weyl 2006-09-03 04:18:00 UTC
SRPM URL: http://home.comcast.net/~ckweyl/perl-Sub-Name-0.02-1.fc5.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-Sub-Name.spec

Description:
This module allows one to "name" or rename subroutines, including anonymous
ones.

Note that this is mainly for aid in debugging; you still cannot call the sub
by the new name (w/o some deep magic).

Comment 1 Patrice Dumas 2006-09-06 08:24:52 UTC
* rpmlint is silent
* package named according to guidelines
* free software, without licence files included
* meets packaging guidelines
* spec legible
* source match upstream
a47e1f1fe88a0c85b7ecc0d150039616  Sub-Name-0.02.tar.gz
* sane provides (providing Name.so is certainly wrong, but should be
  handled automatically by provide script)
Provides: Name.so perl(Sub::Name) = 0.02
* %files right

APPROVED



Comment 2 Patrice Dumas 2006-09-06 08:28:42 UTC
As a side note, it is possible that the -- in the summmary aren't
rendered nicely in all the softs using the summary, maybe they
could be replaced with some ,.

Comment 3 Chris Weyl 2006-09-06 14:57:53 UTC
+Import to CVS
+Add to owners.list
+Bump release, build for devel
+devel build succeeds
+Request branching (FC-5)
+Close bug

Thanks for the review! :)

Comment 4 Chris Weyl 2007-04-19 22:16:15 UTC
Please branch for EL-4, EL-5.

Comment 5 Chris Weyl 2007-04-19 22:16:38 UTC
Please branch for EL-4, EL-5.