Correct Makefile.PL authortests borkage
Peter Rabbitson [Wed, 26 Jan 2011 11:37:50 +0000 (12:37 +0100)]
Fix last-resort error message and do an explicit require (though
it should *not* be necessary people broke it somehow)

Makefile.PL

index e44489b..2a196bc 100644 (file)
@@ -193,14 +193,20 @@ tests_recursive (qw|
 # temporary(?) until I get around to fix M::I wrt xt/
 # needs Module::Install::AuthorTests
 eval {
+  # this should not be necessary since the autoloader is supposed
+  # to work, but there were reports of it failing
+  require Module::Install::AuthorTests;
   recursive_author_tests (qw/xt/);
   1;
 } || do {
   if ($Module::Install::AUTHOR && ! $args->{skip_author_deps}) {
     my $err = $@;
+
+    # better error message in case of missing dep
     eval { require Module::Install::AuthorTests }
       || die "\nYou need Module::Install::AuthorTests installed to run this Makefile.PL in author mode (or supply --skip_author_deps):\n\n$@\n";
-    die $@;
+
+    die $err;
   }
 };