Red Hat Bugzilla – Bug 197484
Review Request: perl-Class-MakeMethods
Last modified: 2010-09-02 17:16:44 EDT
Spec URL: http://home.comcast.net/~ckweyl/perl-Class-MakeMethods.spec
SRPM URL: http://home.comcast.net/~ckweyl/perl-Class-MakeMethods-1.01-0.fc5.src.rpm
The Class::MakeMethods framework allows Perl class developers to quickly
define common types of methods. When a module uses Class::MakeMethods or one
of its subclasses, it can select from a variety of supported method types, and
specify a name for each method desired. The methods are dynamically generated
and installed in the calling package.
Construction of the individual methods is handled by subclasses. This
delegation approach allows for a wide variety of method-generation techniques
to be supported, each by a different subclass. Subclasses can also be added to
provide support for new types of methods.
Over a dozen subclasses are available, including implementations of a variety
of different method-generation techniques. Each subclass generates several
types of methods, with some supporting their own open-eneded extension syntax,
for hundreds of possible combinations of method types.
+ tarballs in SRPM match upstream version.
+ rpmlint doensn't complaints source rpm.
+ Local Build works fine.
+ Mock build works fine.
+ rpmlint doesn't complaints binary rpm.
+ License is the same as perl.
- perl should not be included
- Package doesn't contain verbatin license text.
Jochen, are you reviewing this? You should block FE-REVIEW instead of FE-NEW is
Note that BR: perl is in the optional list; generally we don't see it as a
blocker. I can't tell if you're inticating that it's a blocker or not.
Also note that it is quite rare for a Perl module to include the verbatim
license text, and it not a blocker to fail to include the license text in the
package when the upstream tarball does not include it.
Just some friendly comments. Thanks for helping out with the reviews.
Thank you for your clarification.
After the clarification from Jason, I will be able to APPROVE you package.
Imported and built for devel; will build for FC- after branching.
Thanks for the review!
Is this actually Approved or still waiting for approval?
Given the blocker bug was switched from FE-REVIEW to FE-ACCEPT, I understood
comment #3 to be an approval.
Package Change Request
Package Name: perl-Class-MakeMethods
New Branches: el6
Owners: iarnell tremble
Git done (by process-git-requests).