X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=make_ext.pl;h=de26d8410a6221ad7b4412984f3710ac2c03fc2c;hb=3387927231e5bf4138a3082ecafb247e03a8061a;hp=fb4399cba7a1ef7c913288863f6df4b429fc250d;hpb=ca5de986dd6a53bf639708f4ed28e2bdeb1b3aa8;p=p5sagit%2Fp5-mst-13.2.git diff --git a/make_ext.pl b/make_ext.pl index fb4399c..de26d84 100644 --- a/make_ext.pl +++ b/make_ext.pl @@ -2,20 +2,85 @@ use strict; use warnings; use Config; +BEGIN { + if ($^O eq 'MSWin32') { + unshift @INC, ('../cpan/Cwd', '../cpan/Cwd/lib'); + require File::Spec::Functions; + require FindExt; + } + else { + unshift @INC, 'cpan/Cwd'; + } +} +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/ +# needed to build the nonxs modules +# After which, all nonxs modules are in lib, which was always sufficient to +# allow miniperl to build everything else. + +# 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) . ')'; # This script acts as a simple interface for building extensions. -# It primarily used by the perl Makefile: + +# It's actually a cut and shut of the Unix version ext/utils/makeext and the +# Windows version win32/build_ext.pl hence the two invocation styles. + +# On Unix, it primarily used by the perl Makefile one extention at a time: # # d_dummy $(dynamic_ext): miniperl preplibrary FORCE # @$(RUN) ./miniperl make_ext.pl --target=dynamic $@ MAKE=$(MAKE) LIBPERL_A=$(LIBPERL) # +# On Windows or VMS, +# If '--static' is specified, static extensions will be built. +# If '--dynamic' is specified, dynamic extensions will be built. +# If '--nonxs' is specified, nonxs extensions will be built. +# If '--dynaloader' is specificied, DynaLoader will be built. +# If '--all' is specified, all extensions will be built. +# +# make_ext.pl "MAKE=make [-make_opts]" --dir=directory [--target=target] [--static|--dynamic|--all] +ext2 !ext1 +# +# E.g. +# +# make_ext.pl "MAKE=nmake -nologo" --dir=..\ext +# +# make_ext.pl "MAKE=nmake -nologo" --dir=..\ext --target=clean +# +# make_ext.pl MAKE=dmake --dir=..\ext +# +# make_ext.pl MAKE=dmake --dir=..\ext --target=clean +# +# Will skip building extensions which are marked with an '!' char. +# Mostly because they still not ported to specified platform. +# +# If any extensions are listed with a '+' char then only those +# extensions will be built, but only if they arent countermanded +# by an '!ext' and are appropriate to the type of building being done. + # 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; - my (%excl, %incl, %opts, @extspec, @pass_through); foreach (@ARGV) { @@ -26,7 +91,7 @@ foreach (@ARGV) { } elsif (/^--([\w\-]+)$/) { $opts{$1} = 1; } elsif (/^--([\w\-]+)=(.*)$/) { - $opts{$1} = $2; + push @{$opts{$1}}, $2; } elsif (/=/) { push @pass_through, $_; } elsif (length) { @@ -34,6 +99,11 @@ foreach (@ARGV) { } } +my $static = $opts{static} || $opts{all}; +my $dynamic = $opts{dynamic} || $opts{all}; +my $nonxs = $opts{nonxs} || $opts{all}; +my $dynaloader = $opts{dynaloader} || $opts{all}; + # The Perl Makefile.SH will expand all extensions to # lib/auto/X/X.a (or lib/auto/X/Y/Y.a if nested) # A user wishing to run make_ext might use @@ -42,18 +112,18 @@ foreach (@ARGV) { # canonise into X/Y form (pname) foreach (@extspec) { - if (/^lib/) { + if (s{^lib/auto/}{}) { # Remove lib/auto prefix and /*.* suffix - s{^lib/auto/}{}; - s{[^/]*\.[^/]*$}{}; - } elsif (/^ext/) { + s{/[^/]+\.[^/]+$}{}; + } elsif (s{^$ext_dirs_re/}{}) { # Remove ext/ prefix and /pm_to_blib suffix - s{^ext/}{}; s{/pm_to_blib$}{}; - } elsif (/::/) { + # Targets are given as files on disk, but the extension spec is still + # written using /s for each :: + tr!-!/!; + } elsif (s{::}{\/}g) { # Convert :: to / - s{::}{\/}g; - } elsif (/\..*o$/) { + } else { s/\..*o//; } } @@ -61,7 +131,8 @@ foreach (@extspec) { my $makecmd = shift @pass_through; # Should be something like MAKE=make unshift @pass_through, 'PERL_CORE=1'; -my $target = $opts{target}; +my @dirs = @{$opts{dir} || \@ext_dirs}; +my $target = $opts{target}[0]; $target = 'all' unless defined $target; # Previously, $make was taken from config.sh. However, the user might @@ -75,14 +146,14 @@ unless(defined $makecmd and $makecmd =~ /^MAKE=(.*)$/) { die "$0: WARNING: Please include MAKE=\$(MAKE) in \@ARGV\n"; } -my $make = $1 || $Config{make} || $ENV{MAKE}; +# This isn't going to cope with anything fancy, such as spaces inside command +# names, but neither did what it replaced. Once there is a use case that needs +# it, please supply patches. Until then, I'm sticking to KISS +my @make = split ' ', $1 || $Config{make} || $ENV{MAKE}; # Using an array of 0 or 1 elements makes the subsequent code simpler. my @run = $Config{run}; @run = () if not defined $run[0] or $run[0] eq ''; -if (!@extspec) { - die "$0: no extension specified\n"; -} if ($target eq '') { die "make_ext: no make target specified (eg all or clean)\n"; @@ -91,34 +162,207 @@ if ($target eq '') { die "$0: unknown make target '$target'\n"; } -foreach my $pname (@extspec) { - my $mname = $pname; +if (!@extspec and !$static and !$dynamic and !$nonxs and !$dynaloader) { + die "$0: no extension specified\n"; +} + +my $perl; +my %extra_passthrough; + +if ($is_Win32) { + (my $here = getcwd()) =~ s{/}{\\}g; + $perl = $^X; + if ($perl =~ m#^\.\.#) { + $perl = "$here\\$perl"; + } + (my $topdir = $perl) =~ s/\\[^\\]+$//; + # miniperl needs to find perlglob and pl2bat + $ENV{PATH} = "$topdir;$topdir\\win32\\bin;$ENV{PATH}"; + my $pl2bat = "$topdir\\win32\\bin\\pl2bat"; + unless (-f "$pl2bat.bat") { + my @args = ($perl, "-I$topdir\\lib", ("$pl2bat.pl") x 2); + print "@args\n"; + system(@args) unless defined $::Cross::platform; + } + + my $build = getcwd(); + print "In $build"; + foreach my $dir (@dirs) { + chdir($dir) or die "Cannot cd to $dir: $!\n"; + (my $ext = getcwd()) =~ s{/}{\\}g; + FindExt::scan_ext($ext); + FindExt::set_static_extensions(split ' ', $Config{static_ext}); + chdir $build + or die "Couldn't chdir to '$build': $!"; # restore our start directory + } + + my @ext; + push @ext, FindExt::static_ext() if $static; + push @ext, FindExt::dynamic_ext() if $dynamic; + push @ext, FindExt::nonxs_ext() if $nonxs; + push @ext, 'DynaLoader' if $dynaloader; + + foreach (sort @ext) { + if (%incl and !exists $incl{$_}) { + #warn "Skipping extension $_, not in inclusion list\n"; + next; + } + if (exists $excl{$_}) { + warn "Skipping extension $_, not ported to current platform"; + next; + } + push @extspec, $_; + if($_ eq 'DynaLoader' and $target !~ /clean$/) { + # No, we don't know why nmake can't work out the dependency chain + push @{$extra_passthrough{$_}}, 'DynaLoader.c'; + } elsif(FindExt::is_static($_)) { + push @{$extra_passthrough{$_}}, 'LINKTYPE=static'; + } + } + + chdir '..' + or die "Couldn't chdir to build directory: $!"; # now in the Perl build +} +elsif ($is_VMS) { + $perl = $^X; + push @extspec, (split ' ', $Config{static_ext}) if $static; + push @extspec, (split ' ', $Config{dynamic_ext}) if $dynamic; + push @extspec, (split ' ', $Config{nonxs_ext}) if $nonxs; + push @extspec, 'DynaLoader' if $dynaloader; +} + +{ + # Cwd needs to be built before Encode recurses into subdirectories. + # This seems to be the simplest way to ensure this ordering: + my (@first, @other); + foreach (@extspec) { + if ($_ eq 'Cwd') { + push @first, $_; + } else { + push @other, $_; + } + } + @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 $depth = $pname; - $depth =~ s![^/]+!..!g; - # Always need one more .. for ext/ - my $up = "../$depth"; - my $perl = "$up/miniperl"; + my $ext_pathname; - if ($Config{osname} eq 'catamount') { - # Snowball's chance of building extensions. - die "This is $Config{osname}, not building $mname, sorry.\n"; + # 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 (-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; + } } print "\tMaking $mname ($target)\n"; - build_extension('ext', "ext/$pname", $up, $perl, "$up/lib", - \@pass_through); + build_extension($ext_pathname, $perl, $mname, + [@pass_through, @{$extra_passthrough{$spec} || []}]); } sub build_extension { - my ($ext, $ext_dir, $return_dir, $perl, $lib_dir, $pass_through) = @_; + my ($ext_dir, $perl, $mname, $pass_through) = @_; + unless (chdir "$ext_dir") { warn "Cannot cd to $ext_dir: $!"; return; } + + my $up = $ext_dir; + $up =~ s![^/]+!..!g; + + $perl ||= "$up/miniperl"; + my $return_dir = $up; + my $lib_dir = "$up/lib"; + # $lib_dir must be last, as we're copying files into it, and in a parallel + # make there's a race condition if one process tries to open a module that + # another process has half-written. + my @new_inc = ((map {"$up/$_"} @toolchain), $lib_dir); + if ($is_Win32) { + @new_inc = map {File::Spec::Functions::rel2abs($_)} @new_inc; + } + $ENV{PERL5LIB} = join $Config{path_sep}, @new_inc; + $ENV{PERL_CORE} = 1; + # warn $ENV{PERL5LIB}; + + my $makefile; + if ($is_VMS) { + $makefile = 'descrip.mms'; + if ($target =~ /clean$/ + && !-f $makefile + && -f "${makefile}_old") { + $makefile = "${makefile}_old"; + } + } else { + $makefile = 'Makefile'; + } - if (!-f 'Makefile') { + if (!-f $makefile) { + if (!-f 'Makefile.PL') { + print "\nCreating Makefile.PL in $ext_dir for $mname\n"; + # We need to cope well with various possible layouts + my @dirs = split /::/, $mname; + my $leaf = pop @dirs; + my $leafname = "$leaf.pm"; + my $pathname = join '/', @dirs, $leafname; + my @locations = ($leafname, $pathname, "lib/$pathname"); + my $fromname; + foreach (@locations) { + if (-f $_) { + $fromname = $_; + last; + } + } + + unless ($fromname) { + die "For $mname tried @locations in in $ext_dir but can't find source"; + } + my $pod_name; + ($pod_name = $fromname) =~ s/\.pm\z/.pod/; + $pod_name = $fromname unless -e $pod_name; + open my $fh, '>', 'Makefile.PL' + or die "Can't open Makefile.PL for writing: $!"; + print $fh <<"EOM"; +#-*- buffer-read-only: t -*- + +# This Makefile.PL was written by $0. +# It will be deleted automatically by make realclean + +use strict; +use ExtUtils::MakeMaker; + +WriteMakefile( + NAME => '$mname', + VERSION_FROM => '$fromname', + ABSTRACT_FROM => '$pod_name', + realclean => {FILES => 'Makefile.PL'}, +); + +# ex: set ro: +EOM + close $fh or die "Can't close Makefile.PL: $!"; + } print "\nRunning Makefile.PL in $ext_dir\n"; # Presumably this can be simplified @@ -131,11 +375,18 @@ sub build_extension { @cross = '-MCross'; } - my @perl = (@run, $perl, "-I$lib_dir", @cross, 'Makefile.PL', - 'INSTALLDIRS=perl', 'INSTALLMAN3DIR=none', - @$pass_through); - print join(' ', @perl), "\n"; - my $code = system @perl; + my @args = (@cross, 'Makefile.PL'); + if ($is_VMS) { + my $libd = VMS::Filespec::vmspath($lib_dir); + push @args, "INST_LIB=$libd", "INST_ARCHLIB=$libd"; + } else { + push @args, 'INSTALLDIRS=perl', 'INSTALLMAN1DIR=none', + 'INSTALLMAN3DIR=none'; + } + push @args, @$pass_through; + _quote_args(\@args) if $is_VMS; + print join(' ', @run, $perl, @args), "\n"; + my $code = system @run, $perl, @args; warn "$code from $ext_dir\'s Makefile.PL" if $code; # Right. The reason for this little hack is that we're sitting inside @@ -161,12 +412,12 @@ sub build_extension { cd $ext_dir if test ! -f Makefile -a -f Makefile.old; then echo "Note: Using Makefile.old" - make -f Makefile.old $clean_target MAKE=$make @pass_through + make -f Makefile.old $clean_target MAKE='@make' @pass_through else if test ! -f Makefile ; then echo "Warning: No Makefile!" fi - make $clean_target MAKE=$make @pass_through + make $clean_target MAKE='@make' @pass_through fi cd $return_dir EOS @@ -175,20 +426,47 @@ EOS } } - if (not -f 'Makefile') { + if (not -f $makefile) { print "Warning: No Makefile!\n"; } + if ($is_VMS) { + _macroify_passthrough($pass_through); + unshift @$pass_through, "/DESCRIPTION=$makefile"; + } + if (!$target or $target !~ /clean$/) { # Give makefile an opportunity to rewrite itself. # reassure users that life goes on... - my @config = (@run, $make, 'config', @$pass_through); - system @config and print "@config failed, continuing anyway...\n"; + my @args = ('config', @$pass_through); + _quote_args(\@args) if $is_VMS; + system(@run, @make, @args) and print "@run @make @args failed, continuing anyway...\n"; } - my @targ = (@run, $make, $target, @$pass_through); - print "Making $target in $ext_dir\n@targ\n"; - my $code = system @targ; + my @targ = ($target, @$pass_through); + _quote_args(\@targ) if $is_VMS; + print "Making $target in $ext_dir\n@run @make @targ\n"; + my $code = system(@run, @make, @targ); die "Unsuccessful make($ext_dir): code=$code" if $code != 0; chdir $return_dir || die "Cannot cd to $return_dir: $!"; } + +sub _quote_args { + my $args = shift; # must be array reference + + # Do not quote qualifiers that begin with '/'. + map { if (!/^\//) { + $_ =~ s/\"/""/g; # escape C<"> by doubling + $_ = q(").$_.q("); + } + } @{$args} + ; +} + +sub _macroify_passthrough { + my $passthrough = shift; + _quote_args($passthrough); + my $macro = '/MACRO=(' . join(',',@$passthrough) . ')'; + @$passthrough = (); + @$passthrough[0] = $macro; +}