Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Dist-Zilla-App-Command-cover CPAN distribution.

Report information
The Basics
Id: 72144
Status: resolved
Priority: 0/
Queue: Dist-Zilla-App-Command-cover

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

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



Subject: Load dependencies at runtime instead of compile time
Download (untitled) / with headers
text/plain 426b
Currently Dist::Zilla has a flaw: all App::Command classes are loaded during startup even if not used. So DZ::App::Command::cover is loaded (and its dependencies too) even when just "dzil authordeps" (as an example) is run. To avoid this (and so improve DZ startup), it would be better to load dependencies (File::Temp, Path::Class, File::chdir) only on usage in execute(). -- Olivier Mengué - http://perlresume.org/DOLMEN
Download (untitled) / with headers
text/plain 461b
On Thu Nov 03 13:22:55 2011, DOLMEN wrote: Show quoted text
> Currently Dist::Zilla has a flaw: all App::Command classes are loaded > during startup even if not used. So DZ::App::Command::cover is loaded > (and its dependencies too) even when just "dzil authordeps" (as an > example) is run. > > To avoid this (and so improve DZ startup), it would be better to load > dependencies (File::Temp, Path::Class, File::chdir) only on usage in > execute().
Fixed in release 1.101001


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.