Skip Menu |
 

This queue is for tickets about the String-Diff CPAN distribution.

Report information
The Basics
Id: 48064
Status: resolved
Priority: 0/
Queue: String-Diff

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

Bug Information
Severity: Wishlist
Broken in: 0.04
Fixed in: (no value)



Subject: Change included test modules to a Build_Requires
Download (untitled) / with headers
text/plain 733b
Hi there: Your module uses a bunch of test modules (among other things) which are included in inc/. It adds a bit of unnecessary bulk to your package, since some of the included modules are now in Perl's core module distribution. Moreover, it means you won't get the relevant bug fixes for all of those modules. If your intent is just to use those modules for the purpose of testing and not for your module to function (which is what I gather since the modules are in inc/ and not lib/) then you should ideally use the build_requires relationship (provided by newer build systems like Module::Install and Module::Build). That way the modules won't be installed (unless the user explicitly wants to install them) Cheers, Jonathan
thank you for report! this ticket fixed on 0.05.


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.