Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Dist-Zilla-Plugin-PodWeaver CPAN distribution.

Report information
The Basics
Id: 80684
Status: resolved
Priority: 0/
Queue: Dist-Zilla-Plugin-PodWeaver

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

Bug Information
Severity: Normal
Broken in: 3.101641
Fixed in: (no value)



Subject: If weaver.* exists, logger is not correctly passed to Pod::Weaver
If weaver.* exists in Dist::Zilla root directory, this plugin calls Pod::Weaver->new_from_config($arg). Pod::Weaver::new_from_config takes 2 arguments, excluding class name, and the 2nd argument is passed to Pod::Weaver::new_from_config_sequence(). In this case, it is undef because there is only one argument: $arg. logger is set in Pod::Weaver::new_from_config_sequence() according to the argument. Therefore, logger in Dist::ZIlla is not correctly passed to Pod::Weaver. Actually, `dzil listdeps' mutes Dist::Zilla logger but not Pod::Weaver logger in this case. So, unintentional log messages are embedded into dependency list output. I am not sure that what is the best solution. Attached patch, however, fixes this behavior anyway.
Subject: fix.patch
--- /usr/lib/perl5/site_perl/5.14/Dist/Zilla/Plugin/PodWeaver.pm.orig 2012-11-07 22:48:51.872516600 +0900 +++ /usr/lib/perl5/site_perl/5.14/Dist/Zilla/Plugin/PodWeaver.pm 2012-11-07 22:52:56.707997700 +0900 @@ -45,7 +45,7 @@ return Pod::Weaver->new_from_config_sequence($assembler->sequence, $arg); } elsif (@files) { - return Pod::Weaver->new_from_config($arg); + return Pod::Weaver->new_from_config($arg, $arg); } else { return Pod::Weaver->new_with_default_config($arg); }
Thanks, fixed by 6a7318c. -- rjbs