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...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Chris Grau
QA Contact: David Lawrence
URL: http://search.cpan.org/dist/Template-...
Whiteboard:
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:
Environment:
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: 
http://www.auroralinux.org/people/spot/review/Maypole/perl-Template-Plugin-Class.spec

SRPM Name or Url:
http://www.auroralinux.org/people/spot/review/Maypole/perl-Template-Plugin-Class-0.12-2.src.rpm

Description: 

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
Review:

- 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

Needswork:

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
https://www.redhat.com/archives/fedora-extras-list/2005-August/msg01130.html,
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.