Don't require build/test 'recommends' dependencies

Most CPAN clients actually install configure/build/test dependencies, so it
is wasteful to install dependencies only recommended to configure, build or
test the target distribution (especially since they may not be available at
the time the target dist wants them).

diff --git a/Makefile.PL b/Makefile.PL
index ba5090cfbc9eaf9ad49ddb02b4bdff01109fd4e7..5a106a27b2b4fec32b3e98bb5cee91806614dac8 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -17,7 +17,6 @@ my $CONFIGURE_PREREQS = {
 
 my $BUILD_PREREQS = {
  'ExtUtils::MakeMaker' => '0',
- 'Test::Differences'   => '0',
  'Test::More'          => '0',
  'Test::Warn'          => '0',
 };

diff --git a/lib/Task/Devel/Cover/Recommended.pm b/lib/Task/Devel/Cover/Recommended.pm
index 2ef7e3c966ea9f457fdd205806ae5524d82fed75..cd519e3fd482799e33e7c7992d9908289654a432 100644 (file)
--- a/lib/Task/Devel/Cover/Recommended.pm
+++ b/lib/Task/Devel/Cover/Recommended.pm
@@ -47,10 +47,6 @@ L<ExtUtils::MakeMaker>
 
 =item *
 
-L<Test::Differences>
-
-=item *
-
 L<Test::More>
 
 =item *

diff --git a/samples/generate.pl b/samples/generate.pl
index a8332279be776d6cb1eae2c6cc0eef9a38decfa4..456c6598de60ab448c7bcacc4d337c455e2a1382 100644 (file)
--- a/samples/generate.pl
+++ b/samples/generate.pl
@@ -95,6 +95,13 @@ my %eumm_phases = (
  build     => [ qw<build test> ],
  run       => [ qw<runtime>    ],
 );
+my %meta_phase_relationships = (
+ configure => [ qw<requires>                     ],
+ build     => [ qw<requires>                     ],
+ test      => [ qw<requires>                     ],
+ runtime   => [ qw<requires recommends suggests> ],
+);
+
 my %prereqs = (
  configure => {
   'ExtUtils::MakeMaker' => '0',
@@ -112,7 +119,7 @@ for my $eumm_phase (keys %eumm_phases) {
 
  for my $meta_phase (@{$eumm_phases{$eumm_phase}}) {
 
-  for my $type (qw<requires recommends suggests>) {
+  for my $type (@{$meta_phase_relationships{$meta_phase}}) {
    my $phase_prereqs = $meta->{prereqs}{$meta_phase}{$type};
    next unless $phase_prereqs;