Skip Menu |

This queue is for tickets about the Module-Install-ReadmeFromPod CPAN distribution.

Report information
The Basics
Id: 99050
Status: resolved
Priority: 0/
Queue: Module-Install-ReadmeFromPod

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

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



Subject: uses IO::All and Capture::Tiny when not in admin mode
By default the module loads IO::All and Capture::Tiny whether or not it's in admin mode. Those are only required in admin mode. This causes problems when a user installs a distribution using it, as they may not have those modules installed. See for example: http://www.cpantesters.org/cpan/report/13e86040-4213-11e4-9533-b131854e61d6. Output from '/home/njh/perl5/perlbrew/perls/perl-5.16.2/bin/perl Makefile.PL': Can't locate IO/All.pm in @INC (@INC contains: inc /tmp/CPAN-Reporter-lib-8XI_ /home/njh/perl5/perlbrew/perls/perl-5.16.2/lib/site_perl/5.16.2/x86_64-linux-thread-multi /home/njh/perl5/perlbrew/perls/perl-5.16.2/lib/site_perl/5.16.2 /home/njh/perl5/perlbrew/perls/perl-5.16.2/lib/5.16.2/x86_64-linux-thread-multi /home/njh/perl5/perlbrew/perls/perl-5.16.2/lib/5.16.2 .) at inc/Module/Install/ReadmeFromPod.pm line 8. BEGIN failed--compilation aborted at inc/Module/Install/ReadmeFromPod.pm line 8. Compilation failed in require at inc/Module/Install.pm line 307. String found where operator expected at Makefile.PL line 9, near "readme_from 'lib/Text/Template/LocalVars.pm'" (Do you need to predeclare readme_from?) String found where operator expected at Makefile.PL line 10, near "readme_from 'lib/Text/Template/LocalVars.pm'" (Do you need to predeclare readme_from?) syntax error at Makefile.PL line 9, near "readme_from 'lib/Text/Template/LocalVars.pm'" Execution of Makefile.PL aborted due to compilation errors. https://github.com/djerius/module-install-readmefrompod provides a possible fix.
OOOOPS. My apologies. This report is against unreleased code, not released code. I got confused. Please disregard the report (but please look at the patch)
On Mon Sep 22 10:32:29 2014, DJERIUS wrote: Show quoted text
> OOOOPS. My apologies. This report is against unreleased code, not > released code. I got confused. > > Please disregard the report (but please look at the patch)
Monday morning strikes again. Here's the correct URL: https://github.com/djerius/module-install-readmefrompod/tree/namespace
Thanks for accepting the patch on github and releasing!