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
Bug 166202 - Review Request: perl-Template-Plugin-Class : Allow calling of class methods on arbitrary classes
Summary: Review Request: perl-Template-Plugin-Class : Allow calling of class methods o...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Chris Grau
QA Contact: David Lawrence
Depends On:
Blocks: FE-ACCEPT 166203
TreeView+ depends on / blocked
Reported: 2005-08-17 20:50 UTC by Tom "spot" Callaway
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2005-08-18 04:23:14 UTC
Type: ---

Attachments (Terms of Use)

Description Tom "spot" Callaway 2005-08-17 20:50:18 UTC
Spec Name or Url:

SRPM Name or Url:


Allow calling of class methods on arbitrary classes

(NOTE: This package is one of the Maypole dependencies)

Comment 1 Chris Grau 2005-08-17 22:40:46 UTC

- rpmlint clean
- good package name
- good spec file name
- good license
- license matches upstream
- spec is am. english (and legible)
- source matches upstream (and is latest version)
- package builds on FC-4
- no locales
- no shared libs
- not relocatable
- no duplicate %files
- file permissions okay
- %clean okay
- consistent use of macros
- code, not content
- no -docs package
- %doc okay
- no -devel package


I was prompted to install Module::Build when running rpmbuild.  This should be
in the BR.

> BuildRequires: perl >= 1:5.6.1

Is this BR necessary?

make does not include %{?_smp_mflags}.

> $ rpm -qf /usr/lib/perl5/vendor_perl/5.8.6/Template
> file /usr/lib/perl5/vendor_perl/5.8.6/Template is not owned by any package

perl-Template-Toolkit installs to %{perl_vendorarch} instead of
%{perl_vendorlib}, so it doesn't create/own this Template directory.

RPM doesn't automatically pick up the requires on perl(Template).  Should
probably be added explicitly.

Comment 3 Chris Grau 2005-08-18 03:35:54 UTC
Looks good.  Approved.

Comment 4 Chris Grau 2005-08-18 04:27:00 UTC
Oops.  As mentioned by Ralf Corsepius in,
the explicit requires should be perl(Template::Plugin).  It's provided by the
same package as perl(Template), so it doesn't change anything, but it's good to
be pedantic.

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