Permalink
Browse files

Restore the VMS Test::Harness prereq

From @dakkar:
"@schwern the vms prereq should come after the big 'if', right? gets clobbered otherwise"
  • Loading branch information...
1 parent 78f8c8b commit 7de24f30964142b68365e14fe16d314780ed0c0d @schwern schwern committed Oct 27, 2011
Showing with 7 additions and 6 deletions.
  1. +7 −6 Makefile.PL
View
13 Makefile.PL
@@ -45,14 +45,10 @@ are upgrading from a version older than $LAST_API_CHANGE.
CHANGE_WARN
}
-# Test::Harnesses prior to 2.00 shoved all of @INC onto the command line
-# when a test had -T. This made it too long. So we need a Test::Harness
-# > 2.00 on VMS for t/testlib.t
-my %Extra_Prereqs = ();
-$Extra_Prereqs{'Test::Harness'} = 2.00 if $^O eq 'VMS';
-
check_environment();
+my %Extra_Prereqs = ();
+
# Special case for MakeMaker being built as a vendor package
if( $BUILDING_AS_PACKAGE ) {
# Some of these are lower than what we bundle. That's ok, we
@@ -78,6 +74,11 @@ else {
my::bundles::copy_bundles("bundled", "inc");
}
+# Test::Harnesses prior to 2.00 shoved all of @INC onto the command line
+# when a test had -T. This made it too long. So we need a Test::Harness
+# > 2.00 on VMS for t/testlib.t
+$Extra_Prereqs{'Test::Harness'} = 2.00 if $^O eq 'VMS';
+
my $MM = WriteMakefile(
NAME => $PACKAGE,
VERSION_FROM => "lib/$PACKAGE_FILE.pm", # finds $VERSION

0 comments on commit 7de24f3

Please sign in to comment.