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