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 bugzilla.redhat.com.
Bug 166197 - Review Request: perl-Class-Trigger : Mixin to add / call inheritable triggers
Summary: Review Request: perl-Class-Trigger : Mixin to add / call inheritable triggers
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/Class-Tri...
Whiteboard:
Depends On:
Blocks: FE-ACCEPT 166184
TreeView+ depends on / blocked
 
Reported: 2005-08-17 20:43 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:24:28 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Tom "spot" Callaway 2005-08-17 20:43:35 UTC
Spec Name or Url: 
http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-Trigger.spec

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

Description: 

Mixin to add / call inheritable triggers

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

Comment 1 Chris Grau 2005-08-17 22:04:09 UTC
Review:

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

Needswork:

BuildRequires: perl >= 1:5.6.1

Is this necessary?  Perl is a given for rpm-build and Class::Trigger doesn't
explicitly require a minimum version.

Makefile.PL lists Class::Data::Inheritable >= 0.02 as a requirement, but it's
not listed as a BR (nor does it build without it installed).

In %build, make is given without %{_smp_mflags}.

Comment 2 Tom "spot" Callaway 2005-08-17 22:32:30 UTC
The BuildRequires: perl >= 1:5.6.1 is in the spec template... but I doubt we
need it.

(perl spec template here: http://gsd.di.uminho.pt/jpo/perl/specfiles/)

I didn't see any reason to use _smp_mflags for a noarch perl package that just
runs a cp command when make is triggered... but for consistency, I've added it. :)

Good catch on C::D::I, i've added it as a BR.

New SRPM:
http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-Trigger-0.09-3.src.rpm

New SPEC:
http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-Trigger.spec

Comment 3 Chris Grau 2005-08-17 22:47:50 UTC
(In reply to comment #2)
> The BuildRequires: perl >= 1:5.6.1 is in the spec template... but I doubt we
> need it.
> 
> (perl spec template here: http://gsd.di.uminho.pt/jpo/perl/specfiles/)

I used to use the same BR for perl.  The spectemplate-perl.spec in the
fedora-rpmdevtools package only lists "BuildRequires: perl."  I think it's
redundant, but probably there for looks in a perl-* package.  I noticed that
perl is already a requirement for rpm-build, so the whole thing is pretty silly.

> I didn't see any reason to use _smp_mflags for a noarch perl package that just
> runs a cp command when make is triggered... but for consistency, I've added it. :)

Agreed, it's pointless in such a case.  I would have approved either way. :)

> Good catch on C::D::I, i've added it as a BR.
> 
> New SRPM:
>
http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-Trigger-0.09-3.src.rpm
> 
> New SPEC:
> http://www.auroralinux.org/people/spot/review/Maypole/perl-Class-Trigger.spec

And with that, APPROVED.

Comment 4 Ville Skyttä 2005-08-18 05:42:32 UTC
perl >= 1:5.6.1 was originally in the spec template because vendor install 
dirs are not supported for earlier versions.  The version has been removed in 
newer versions because all old hacks that would be needed on FC1 and earlier 
has been pruned from the template, too -> it serves no purpose any more. 
 
I'm not convinced that perl will always be pulled in by rpm-build in the 
future because it appears to be used for, duh, mainly macros related to perl 
packages only.  So unless someone can point out a real world problem with it, 
I'm going to keep the explicit "BuildRequires: perl" in the perl packages I 
maintain as well as the spec template in fedora-rpmdevtools. 


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