Skip Menu |
 

This queue is for tickets about the Catalyst-Log-Log4perl CPAN distribution.

Report information
The Basics
Id: 54676
Status: new
Priority: 0/
Queue: Catalyst-Log-Log4perl

People
Owner: Nobody in particular
Requestors: MSCHILLI [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.04
Fixed in: (no value)

Attachments
0001-Rewrote-large-parts-got-rid-of-layout-cspec-manglin.patch



Subject: Fix for broken caller()-based cspecs handling
Download (untitled) / with headers
text/plain 784b
Here's a patch which rewrites large parts of the module in order to use native Log4perl features instead of the 'experimental' code working around broken caller()-based cspecs. To support Catalyst's idea of logging, which happens either immediately on 'autoflush' or manually on 'flush' or not at all on 'abort', the code now takes an existing Log4perl configuration, and adds a Buffer appender in between every appender defined so far. I think it's much cleaner and much less code than the current implementation. Note that it requires Log4perl 1.28, which hasn't been released yet, but you can fetch the tarball from here: http://github.com/mschilli/log4perl/tarball/test_128 Let me know if it works for you and Log4perl 1.28 can be released shortly after. (Patch attached).
Subject: 0001-Rewrote-large-parts-got-rid-of-layout-cspec-manglin.patch

Message body is not shown because it is too large.



This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.