X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=make_ext.pl;h=de26d8410a6221ad7b4412984f3710ac2c03fc2c;hb=bc8f2ddd12bfb4ed7885096cdab471dc8d1188aa;hp=7734bf585ac789d2721f2c99bd7c3b08e9c2daf4;hpb=2d53619e84b23c168ecb5cfd35e6cfb10dd066d0;p=p5sagit%2Fp5-mst-13.2.git

diff --git a/make_ext.pl b/make_ext.pl
index 7734bf5..de26d84 100644
--- a/make_ext.pl
+++ b/make_ext.pl
@@ -6,6 +6,7 @@ BEGIN {
     if ($^O eq 'MSWin32') {
 	unshift @INC, ('../cpan/Cwd', '../cpan/Cwd/lib');
 	require File::Spec::Functions;
+	require FindExt;
     }
     else {
 	unshift @INC, 'cpan/Cwd';
@@ -13,6 +14,10 @@ BEGIN {
 }
 use Cwd;
 
+my $is_Win32 = $^O eq 'MSWin32';
+my $is_VMS = $^O eq 'VMS';
+my $is_Unix = !$is_Win32 && !$is_VMS;
+
 # To clarify, this isn't the entire suite of modules considered "toolchain"
 # It's not even all modules needed to build ext/
 # It's just the source paths of the (minimum complete set of) modules in ext/
@@ -20,11 +25,19 @@ use Cwd;
 # After which, all nonxs modules are in lib, which was always sufficient to
 # allow miniperl to build everything else.
 
-my @toolchain = qw(ext/constant/lib cpan/Cwd cpan/Cwd/lib
-		   ext/ExtUtils-Command/lib
-		   dist/ExtUtils-Install/lib ext/ExtUtils-MakeMaker/lib
-		   ext/ExtUtils-Manifest/lib ext/Text-ParseWords/lib
-		   cpan/File-Path/lib cpan/AutoLoader/lib);
+# This list cannot get any longer without overflowing the length limit for
+# environment variables on VMS
+my @toolchain = qw(cpan/AutoLoader/lib
+		   cpan/Cwd cpan/Cwd/lib
+		   cpan/ExtUtils-Command/lib
+		   dist/ExtUtils-Install/lib
+		   cpan/ExtUtils-MakeMaker/lib
+		   cpan/ExtUtils-Manifest/lib
+		   cpan/File-Path/lib
+		   );
+
+# Used only in ExtUtils::Liblist::Kid::_win32_ext()
+push @toolchain, 'cpan/Text-ParseWords/lib' if $is_Win32;
 
 my @ext_dirs = qw(cpan dist ext);
 my $ext_dirs_re = '(?:' . join('|', @ext_dirs) . ')';
@@ -68,12 +81,6 @@ my $ext_dirs_re = '(?:' . join('|', @ext_dirs) . ')';
 # It may be deleted in a later release of perl so try to
 # avoid using it for other purposes.
 
-my $is_Win32 = $^O eq 'MSWin32';
-my $is_VMS = $^O eq 'VMS';
-my $is_Unix = !$is_Win32 && !$is_VMS;
-
-require FindExt if $is_Win32;
-
 my (%excl, %incl, %opts, @extspec, @pass_through);
 
 foreach (@ARGV) {
@@ -238,37 +245,36 @@ elsif ($is_VMS) {
     @extspec = (@first, @other);
 }
 
+if ($Config{osname} eq 'catamount' and @extspec) {
+    # Snowball's chance of building extensions.
+    die "This is $Config{osname}, not building $extspec[0], sorry.\n";
+}
+
 foreach my $spec (@extspec)  {
     my $mname = $spec;
     $mname =~ s!/!::!g;
     my $ext_pathname;
-    if (-d "ext/$spec"
-	# Temporary hack to cope with smokers that are not clearing directories:
-	&& $spec =~ m!/!
-       ) {
-	# Old style ext/Data/Dumper/
-	$ext_pathname = "ext/$spec";
-    } else {
-	# New style ext/Data-Dumper/
-	my $copy = $spec;
-	$copy =~ tr!/!-!;
-	foreach my $dir (@ext_dirs) {
-	    if (-d "$dir/$copy") {
-		$ext_pathname = "$dir/$copy";
-		last;
-	    }
+
+    # Try new style ext/Data-Dumper/ first
+    my $copy = $spec;
+    $copy =~ tr!/!-!;
+    foreach my $dir (@ext_dirs) {
+	if (-d "$dir/$copy") {
+	    $ext_pathname = "$dir/$copy";
+	    last;
 	}
-	if (!defined $ext_pathname) {
+    }
+
+    if (!defined $ext_pathname) {
+	if (-d "ext/$spec") {
+	    # Old style ext/Data/Dumper/
+	    $ext_pathname = "ext/$spec";
+	} else {
 	    warn "Can't find extension $spec in any of @ext_dirs";
 	    next;
 	}
     }
 
-    if ($Config{osname} eq 'catamount') {
-	# Snowball's chance of building extensions.
-	die "This is $Config{osname}, not building $mname, sorry.\n";
-    }
-
     print "\tMaking $mname ($target)\n";
 
     build_extension($ext_pathname, $perl, $mname,
@@ -298,6 +304,7 @@ sub build_extension {
     }
     $ENV{PERL5LIB} = join $Config{path_sep}, @new_inc;
     $ENV{PERL_CORE} = 1;
+    # warn $ENV{PERL5LIB};
 
     my $makefile;
     if ($is_VMS) {