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

[José Pedro Oliveira]

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. 

I disagree with your opinion and I believe my decision is correct.

Reasoning:

1) to create an ApacheLog dispatcher you need to supply an Apache
   object (see
http://search.cpan.org/dist/Log-Dispatch/lib/Log/Dispatch/ApacheLog.pm)

2) you can only obtain an Apache* object if your app is running
   under mod_perl

So, your app should be the one requiring mod_perl and not
perl-Log-Dispatch.

Also, when installing this perl package using CPAN you are not
required to install mod_perl (see Build.PL).

This kind of requirements nightmare can even get worst if we
start to talk about Log::Log4perl which uses Log::Dispatch.

> 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.

This could be a solution if there are packages that need
perl-Log-Dispatch to install mod_perl and apache (I never
came across one).


jpo
-- 
José Pedro Oliveira
* mailto: jpo at di.uminho.pt * http://gsd.di.uminho.pt/jpo *
* gpg fingerprint = F9B6 8D87 859D 1C94 48F0 84C0 9749 9EB5 91BD 851B *
http://conferences.yapceurope.org/2005/ * http://braga.yapceurope.org/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 251 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/fedora-extras-commits/attachments/20050924/19d39abb/attachment-0001.sig>