Skip Menu |

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

Report information
The Basics
Id: 110915
Status: resolved
Priority: 0/
Queue: Log-Log4perl

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

Bug Information
Severity: (no value)
Broken in: 1.46
Fixed in: (no value)



Subject: Redundant argument in sprintf warnings in Log::Log4perl::DateFormat with Perl 5.22
Redundant argument in sprintf at /home/foo/perl5/perlbrew/perls/22/lib/site_perl/5.22.1/Log/Log4perl/DateFormat.pm line 305. The line in question is: https://metacpan.org/source/MSCHILLI/Log-Log4perl-1.46/lib/Log/Log4perl/DateFormat.pm#L305 My output format is: log4perl.appender.SCREEN "Log::Log4perl::Appender::ScreenColoredLevels" log4perl.appender.SCREEN.layout "Log::Log4perl::Layout::PatternLayout" log4perl.appender.SCREEN.layout.ConversionPattern "%d{yyyy-MM-dd HH:mm:ss.SSS Z} %-5p %X{hostname} (%X{alias}) %m%n" The warning was added to Perl in this commit: http://perl5.git.perl.org/perl.git/commit/4077a6bc0ae42279f757dffc08ee68ba8ace9924
Thanks for the report! I went ahead and fixed the problem in https://github.com/mschilli/log4perl/commit/fae51729ac219449f5d061cc903eb2a4352225a5 and it'll go out with the next release (1.47, hopefully soon).
Thanks for the quick fix! I did look at the code but wasn't able to identify the cause. My workaround was a no warnings 'redundant' just before the sprintf.
When will you release a version which includes this fix?
On Tue Mar 15 13:51:29 2016, ABRAXXA wrote: Show quoted text
> When will you release a version which includes this fix?
Just released 1.47 to CPAN which contains the fix, sorry for the delay!
Thank you very much!