[Catalyst] Catalyst::Log::Log4perl branch

Adam Jacob adam at stalecoffee.org
Fri Mar 2 19:26:22 GMT 2007


Several people have pointed out that various parts of Log::Log4perl  
don't work well with Catalyst.  Specifically, many of the options  
that rely on Stack information are incorrect.  The format strings:

%L Line number within the file where the log statement was issued
%F File where the logging event occurred
%C Fully qualified package (or class) name of the caller
%M Method or function where the logging request was issued
%l Fully qualified name of the calling method followed by the callers  
source the file name and line number between parentheses.

Sebastian Willert has helpfully provided a patch that should resolve  
these issues.  He rules.  His changes have been put on this branch:

http://dev.catalyst.perl.org/repos/Catalyst/branches/Catalyst-Log- 
Log4perl/cspec-fixes

I'm going to be pretty busy for the next few days, but don't want to  
sit on this any longer.  Can those of you using  
Catalyst::Log::Log4perl give this a whirl, and make sure it works  
with your applications? (And doesn't cause any kind of nasty  
performance implications, especially with the cspec fixes disabled?)

Your rewards will be my enduring gratitude, and a better  
Catalyst::Log::Log4perl.

Thanks!

Adam



More information about the Catalyst mailing list