X-Git-Url: https://perl5.git.perl.org/perl5.git/blobdiff_plain/9822e6e0ce235da7838e955762323f2bb3477e07..f723540e14965a776d0549743a746fd3b1069fc5:/make_ext.pl diff --git a/make_ext.pl b/make_ext.pl index d0c6328..3254628 100644 --- a/make_ext.pl +++ b/make_ext.pl @@ -3,16 +3,53 @@ use strict; use warnings; use Config; +my $is_Win32 = $^O eq 'MSWin32'; +my $is_VMS = $^O eq 'VMS'; +my $is_Unix = !$is_Win32 && !$is_VMS; + +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 extension 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 specified, 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 (%excl, %incl, %opts, @extspec, @passthrough); +my (%excl, %incl, %opts, @extspec, @pass_through); foreach (@ARGV) { if (/^!(.*)$/) { @@ -22,49 +59,18 @@ foreach (@ARGV) { } elsif (/^--([\w\-]+)$/) { $opts{$1} = 1; } elsif (/^--([\w\-]+)=(.*)$/) { - $opts{$1} = $2; - } elsif (/^--([\w\-]+)=(.*)$/) { - $opts{$1} = $2; + push @{$opts{$1}}, $2; } elsif (/=/) { - push @passthrough, $_; - } else { + push @pass_through, $_; + } elsif (length) { push @extspec, $_; } } -my $target = $opts{target}; -my $extspec = $extspec[0]; -my $makecmd = shift @passthrough; # Should be something like MAKE=make -my $passthru = join ' ', @passthrough; # allow extra macro=value to be passed through -print "\n"; - -# Previously, $make was taken from config.sh. However, the user might -# instead be running a possibly incompatible make. This might happen if -# the user types "gmake" instead of a plain "make", for example. The -# correct current value of MAKE will come through from the main perl -# makefile as MAKE=/whatever/make in $makecmd. We'll be cautious in -# case third party users of this script (are there any?) don't have the -# MAKE=$(MAKE) argument, which was added after 5.004_03. -my $make; -if (defined($makecmd) and $makecmd =~ /^MAKE=(.*)$/) { - $make = $1; -} -else { - print "ext/util/make_ext: WARNING: Please include MAKE=\$(MAKE)\n"; - print "\tin your call to make_ext. See ext/util/make_ext for details.\n"; - exit(1); -} - -# fallback to config.sh's MAKE -$make ||= $Config{make} || $ENV{MAKE}; -my $run = $Config{run}; -$run = '' if not defined $run; -$run .= ' ' if $run ne '';; - -if (!defined($extspec) or $extspec eq '') { - print "make_ext: no extension specified\n"; - exit(1); -} +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) @@ -73,90 +79,356 @@ if (!defined($extspec) or $extspec eq '') { # canonise into X/Y form (pname) -my $pname = $extspec; -if ($extspec =~ /^lib/) { +foreach (@extspec) { + if (s{^lib/auto/}{}) { # Remove lib/auto prefix and /*.* suffix - $pname =~ s{^lib/auto/}{}; - $pname =~ s{[^/]*\.[^/]*$}{}; -} -elsif ($extspec =~ /^ext/) { + s{/[^/]+\.[^/]+$}{}; + } elsif (s{^$ext_dirs_re/}{}) { # Remove ext/ prefix and /pm_to_blib suffix - $pname =~ s{^ext/}{}; - $pname =~ s{/pm_to_blib$}{}; -} -elsif ($extspec =~ /::/) { + s{/pm_to_blib$}{}; + # Targets are given as files on disk, but the extension spec is still + # written using /s for each :: + tr!-!/!; + } elsif (s{::}{\/}g) { # Convert :: to / - $pname =~ s{::}{\/}g; + } else { + s/\..*o//; + } } -elsif ($extspec =~ /\..*o$/) { - $pname =~ s/\..*o//; + +my $makecmd = shift @pass_through; # Should be something like MAKE=make +unshift @pass_through, 'PERL_CORE=1'; + +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 +# instead be running a possibly incompatible make. This might happen if +# the user types "gmake" instead of a plain "make", for example. The +# correct current value of MAKE will come through from the main perl +# makefile as MAKE=/whatever/make in $makecmd. We'll be cautious in +# case third party users of this script (are there any?) don't have the +# MAKE=$(MAKE) argument, which was added after 5.004_03. +unless(defined $makecmd and $makecmd =~ /^MAKE=(.*)$/) { + die "$0: WARNING: Please include MAKE=\$(MAKE) in \@ARGV\n"; } -my $mname = $pname; -$mname =~ s!/!::!g; -my $depth = $pname; -$depth =~ s![^/]+!..!g; -my $makefile = "Makefile"; -my $makeopts = ''; +# 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 (not -d "ext/$pname") { - print "\tSkipping $extspec (directory does not exist)\n"; - exit(0); # not an error ? +if ($target eq '') { + die "make_ext: no make target specified (eg all or clean)\n"; +} elsif ($target !~ /(?:^all|clean)$/) { + # for the time being we are strict about what make_ext is used for + die "$0: unknown make target '$target'\n"; } -if ($Config{osname} eq 'catamount') { - # Snowball's chance of building extensions. - print "This is $Config{osname}, not building $mname, sorry.\n"; - exit(0); +if (!@extspec and !$static and !$dynamic and !$nonxs and !$dynaloader) { + die "$0: no extension specified\n"; } -print "\tMaking $mname ($target)\n"; +my $perl; +my %extra_passthrough; -chdir("ext/$pname"); +if ($is_Win32) { + require Cwd; + require FindExt; + my $build = Cwd::getcwd(); + $perl = $^X; + if ($perl =~ m#^\.\.#) { + my $here = $build; + $here =~ s{/}{\\}g; + $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; + } + + print "In $build"; + foreach my $dir (@dirs) { + chdir($dir) or die "Cannot cd to $dir: $!\n"; + (my $ext = Cwd::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'; + } + } -# check link type and do any preliminaries. Valid link types are -# 'dynamic', 'static', and 'static_pic' (the last one respects -# CCCDLFLAGS such as -fPIC -- see static_target in the main Makefile.SH) -if ($target eq 'dynamic') { - $passthru = "LINKTYPE=dynamic $passthru"; - $target = 'all'; + chdir '..' + or die "Couldn't chdir to build directory: $!"; # now in the Perl build } -elsif ($target eq 'static') { - $passthru = "LINKTYPE=static CCCDLFLAGS= $passthru"; - $target = 'all'; +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; } -elsif ($target eq 'static_pic') { - $passthru = "LINKTYPE=static $passthru"; - $target = 'all'; + +{ + # Cwd needs to be built before Encode recurses into subdirectories. + # Pod::Simple needs to be built before Pod::Functions + # This seems to be the simplest way to ensure this ordering: + my (@first, @other); + foreach (@extspec) { + if ($_ eq 'Cwd' || $_ eq 'Pod/Simple') { + push @first, $_; + } else { + push @other, $_; + } + } + @extspec = (@first, @other); } -elsif ($target eq 'nonxs') { - $target = 'all'; + +if ($Config{osname} eq 'catamount' and @extspec) { + # Snowball's chance of building extensions. + die "This is $Config{osname}, not building $extspec[0], sorry.\n"; } -elsif ($target =~ /clean$/) { - # If Makefile has been moved to Makefile.old by a make clean - # then use Makefile.old for realclean rather than rebuild it - if (! -f $makefile and -f "Makefile.old") { - $makefile = "Makefile.old"; - $makeopts = "-f $makefile"; - print "Note: Using Makefile.old\n"; + +foreach my $spec (@extspec) { + my $mname = $spec; + $mname =~ s!/!::!g; + my $ext_pathname; + + # 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_pathname, $perl, $mname, + [@pass_through, @{$extra_passthrough{$spec} || []}]); } -elsif ($target eq '') { - print "make_ext: no make target specified (eg static or dynamic)\n"; - exit(1); + +sub build_extension { + 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"; + $ENV{PERL_CORE} = 1; + + 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) { + open my $mfh, $makefile or die "Cannot open $makefile: $!"; + while (<$mfh>) { + # Plagiarised from CPAN::Distribution + last if /MakeMaker post_initialize section/; + next unless /^#\s+VERSION_FROM\s+=>\s+(.+)/; + my $vmod = eval $1; + my $oldv; + while (<$mfh>) { + next unless /^XS_VERSION = (\S+)/; + $oldv = $1; + last; + } + last unless defined $oldv; + require ExtUtils::MM_Unix; + defined (my $newv = parse_version MM $vmod) or last; + if ($newv ne $oldv) { + 1 while unlink $makefile + } + } + } + + if (!-f $makefile) { + if (!-f 'Makefile.PL') { + print "\nCreating Makefile.PL in $ext_dir for $mname\n"; + my ($fromname, $key, $value); + if ($mname eq 'podlators') { + # We need to special case this somewhere, and this is fewer + # lines of code than a core-only Makefile.PL, and no more + # complex + $fromname = 'VERSION'; + $key = 'DISTNAME'; + $value = 'podlators'; + $mname = 'Pod'; + } else { + $key = 'ABSTRACT_FROM'; + # 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"); + foreach (@locations) { + if (-f $_) { + $fromname = $_; + last; + } + } + + unless ($fromname) { + die "For $mname tried @locations in in $ext_dir but can't find source"; + } + ($value = $fromname) =~ s/\.pm\z/.pod/; + $value = $fromname unless -e $value; + } + open my $fh, '>', 'Makefile.PL' + or die "Can't open Makefile.PL for writing: $!"; + printf $fh <<'EOM', $0, $mname, $fromname, $key, $value; +#-*- buffer-read-only: t -*- + +# This Makefile.PL was written by %s. +# It will be deleted automatically by make realclean + +use strict; +use ExtUtils::MakeMaker; + +# This is what the .PL extracts to. Not the ultimate file that is installed. +# (ie Win32 runs pl2bat after this) + +# Doing this here avoids all sort of quoting issues that would come from +# attempting to write out perl source with literals to generate the arrays and +# hash. +my @temps = 'Makefile.PL'; +foreach (glob('scripts/pod*.PL')) { + # The various pod*.PL extractors change directory. Doing that with relative + # paths in @INC breaks. It seems the lesser of two evils to copy (to avoid) + # the chdir doing anything, than to attempt to convert lib paths to + # absolute, and potentially run into problems with quoting special + # characters in the path to our build dir (such as spaces) + require File::Copy; + + my $temp = $_; + $temp =~ s!scripts/!!; + File::Copy::copy($_, $temp) or die "Can't copy $temp to $_: $!"; + push @temps, $temp; } -else { - # for the time being we are strict about what make_ext is used for - print "make_ext: unknown make target '$target'\n"; - exit(1); + +my $script_ext = $^O eq 'VMS' ? '.com' : ''; +my %%pod_scripts; +foreach (glob('pod*.PL')) { + my $script = $_; + s/.PL$/$script_ext/i; + $pod_scripts{$script} = $_; } +my @exe_files = values %%pod_scripts; +WriteMakefile( + NAME => '%s', + VERSION_FROM => '%s', + %-13s => '%s', + realclean => { FILES => "@temps" }, + (%%pod_scripts ? ( + PL_FILES => \%%pod_scripts, + EXE_FILES => \@exe_files, + clean => { FILES => "@exe_files" }, + ) : ()), +); -if (not -f $makefile) { - if (-f "Makefile.PL") { - my $cross = $opts{cross} ? ' -MCross' : ''; - system("${run}../$depth/miniperl -I../$depth/lib$cross Makefile.PL INSTALLDIRS=perl INSTALLMAN3DIR=none PERL_CORE=1 $passthru"); +# ex: set ro: +EOM + close $fh or die "Can't close Makefile.PL: $!"; + # As described in commit 23525070d6c0e51f: + # Push the atime and mtime of generated Makefile.PLs back 4 + # seconds. In certain circumstances ( on virtual machines ) the + # generated Makefile.PL can produce a Makefile that is older than + # the Makefile.PL. Altering the atime and mtime backwards by 4 + # seconds seems to resolve the issue. + eval { + my $ftime = time - 4; + utime $ftime, $ftime, 'Makefile.PL'; + }; } + print "\nRunning Makefile.PL in $ext_dir\n"; + + # Presumably this can be simplified + my @cross; + if (defined $::Cross::platform) { + # Inherited from win32/buildext.pl + @cross = "-MCross=$::Cross::platform"; + } elsif ($opts{cross}) { + # Inherited from make_ext.pl + @cross = '-MCross'; + } + + my @args = ("-I$lib_dir", @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 # a program run by ./miniperl, but there are tasks we need to perform # when the 'realclean', 'distclean' or 'veryclean' targets are run. @@ -168,49 +440,66 @@ if (not -f $makefile) { # some of them rely on a $(PERL) for their own distclean targets. # But this always used to be a problem with the old /bin/sh version of # this. - my $suffix = '.sh'; - foreach my $clean_target ('realclean', 'veryclean') { - my $file = "../$depth/$clean_target$suffix"; + if ($is_Unix) { + my $suffix = '.sh'; + foreach my $clean_target ('realclean', 'veryclean') { + my $file = "$return_dir/$clean_target$suffix"; open my $fh, '>>', $file or die "open $file: $!"; # Quite possible that we're being run in parallel here. # Can't use Fcntl this early to get the LOCK_EX flock $fh, 2 or warn "flock $file: $!"; - if ($^O eq 'VMS') { - # Write out DCL here - } elsif ($^O eq 'MSWin32') { - # Might not need anything here. - } else { - print $fh <<"EOS"; -chdir ext/$pname -if test ! -f $makefile -a -f Makefile.old; then + print $fh <<"EOS"; +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 $passthru + make -f Makefile.old $clean_target MAKE='@make' @pass_through else - if test ! -f $makefile ; then + if test ! -f Makefile ; then echo "Warning: No Makefile!" fi - make $clean_target MAKE=$make $passthru + make $clean_target MAKE='@make' @pass_through fi -chdir ../$depth +cd $return_dir EOS - } close $fh or die "close $file: $!"; + } } -} + } -if (not -f $makefile) { + if (not -f $makefile) { print "Warning: No Makefile!\n"; -} + } -if ($target eq 'clean') { -} -elsif ($target eq 'realclean') { -} -else { + if ($is_VMS) { + _quote_args($pass_through); + @$pass_through = ( + "/DESCRIPTION=$makefile", + '/MACRO=(' . join(',',@$pass_through) . ')' + ); + } + + if (!$target or $target !~ /clean$/) { # Give makefile an opportunity to rewrite itself. # reassure users that life goes on... - system( "$run$make config MAKE=$make $passthru" ) - and print "$make config failed, continuing anyway...\n"; + my @args = ('config', @$pass_through); + system(@run, @make, @args) and print "@run @make @args failed, continuing anyway...\n"; + } + my @targ = ($target, @$pass_through); + 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: $!"; } -system "$run$make $target MAKE=$make $passthru" and exit $?; +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} + ; +}