Bug 168044

Summary: Review Request: perl-Log-Dispatch - Dispatches messages to one or more outputs
Product: [Fedora] Fedora Reporter: Jose Pedro Oliveira <jose.p.oliveira.oss>
Component: Package ReviewAssignee: Ralf Corsepius <rc040203>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://gsd.di.uminho.pt/jpo/software/fedora/perl-Log-Dispatch-2.11-1.src.rpm
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-09-12 19:08:09 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 168043    
Bug Blocks: 163779    

Description Jose Pedro Oliveira 2005-09-11 16:09:45 UTC 
Spec Name or Url:
http://gsd.di.uminho.pt/jpo/software/fedora/perl-Log-Dispatch.spec
SRPM Name or Url:
http://gsd.di.uminho.pt/jpo/software/fedora/perl-Log-Dispatch-2.11-1.src.rpm

Description:
Log::Dispatch is a suite of OO modules for logging messages to
multiple outputs, each of which can have a minimum and maximum log
level.  It is designed to be easily subclassed, both for creating a
new dispatcher object and particularly for creating new outputs.

Note:
This is an RT requirement.
Comment 1 Jose Pedro Oliveira 2005-09-11 16:11:33 UTC 
mod_perl perl script to test Log::Dispatch::ApacheLog:
http://gsd.di.uminho.pt/jpo/software/fedora/apachelog.pl
Comment 2 Ralf Corsepius 2005-09-12 05:55:43 UTC 
One release technical issue:

How do you plan to handle the mod_perl2/mod_perl issue inside of CVS?

I'd recommend to remove the mod_perl2 %define for FC > 4 rpm specs (and to 
"BR: mod_perl > ..."), but to keep it for FC-3 only.
Alternatively, you could consider to check for Apache::Log or Apache::Log inside
of the spec file and dynamically apply the patch, depending on the result of
such a check.

Anyway, final decision left to you.

APPROVED.
Comment 3 Jose Pedro Oliveira 2005-09-12 19:08:09 UTC 
Release 2: Better mod_perl handling. Just using:

  if %{__perl} -e 'use Apache2::Log' ; then
  %patch0 -p1
  fi