rpms/perl-Log-Dispatch/devel perl-Log-Dispatch.spec,1.2,1.3

Tom 'spot' Callaway tcallawa at redhat.com
Fri Sep 23 16:12:25 UTC 2005


On Fri, 2005-09-23 at 09:52 +0200, Ralf Corsepius wrote:
> On Thu, 2005-09-22 at 13:48 -0400, Jose Pedro Oliveira wrote:
> > Author: jpo
> > 
> > Update of /cvs/extras/rpms/perl-Log-Dispatch/devel
> > In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv21370/devel
> > 
> > Modified Files:
> > 	perl-Log-Dispatch.spec 
> > Log Message:
> > - Exclude mod_perl from the requirements list
> >   (overkill for most applications using Log::Dispatch).
> 
> VETO
> 
> IMO, this decision is a fault and should be reverted. You are breaking
> package dependencies. 
> 
> If you find this dependency to be too annoying, you probably better
> should consider to package 
> %perl_vendordir/Log/Dispatch/ApacheLog.pm
> into a separate subpackage.

I agree. If a file in a package needs mod_perl (or any other dependency)
to fully function, then you should NOT be excluding that dependency from
the list. The only time excluding dependencies is ok is if it is a false
positive (rpm says it needs perl, but nothing actually does).

~spot
-- 
Tom "spot" Callaway: Red Hat Senior Sales Engineer || GPG ID: 93054260
Fedora Extras Steering Committee Member (RPM Standards and Practices)
Aurora Linux Project Leader: http://auroralinux.org
Lemurs, llamas, and sparcs, oh my!




More information about the fedora-extras-list mailing list