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)



Subject: Fix for broken caller()-based cspecs handling
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.