Couple of makefile fixes:

use is compile time, use require
recommends is for distro maintainers only, push the dependency into the authors hash (it is not to be executed by mere mortals)

diff --git a/Makefile.PL b/Makefile.PL
index b28f992..044802d 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -42,7 +42,6 @@ requires 'SQL::Abstract::Limit'     => 0.13;
 requires 'Sub::Name'                => 0.04;
 
 recommends 'SQL::Translator'        => 0.09004;
-recommends 'Module::Install::Pod::Inherit' => '0.01';
 
 install_script (qw|
     script/dbicadmin
@@ -62,6 +61,7 @@ resources 'MailingList' => 'http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/db
 
 my %force_requires_if_author = (
   'Test::Pod::Coverage'       => 1.04,
+  'Module::Install::Pod::Inherit' => 0.01,
   'SQL::Translator'           => 0.09007,
 
   # CDBI-compat related
@@ -136,14 +136,13 @@ EOW
     print "Removing MANIFEST\n";
     unlink 'MANIFEST';
   }
+
+  eval { require Module::Install::Pod::Inherit };
+  Module::Install::Pod::Inherit::PodInherit() if !$@;
 }
 
 auto_install();
 
-eval { use Module::Install::Pod::Inherit; };
-
-PodInherit() if(!$@ && $Module::Install::AUTHOR);
-
 WriteAll();
 
 # Re-write META.yml to _exclude_ all forced requires (we do not want to ship this)