Permissions in MakeMaker (Was: patch to MM_Unix.pm)
[p5sagit/p5-mst-13.2.git] / lib / ExtUtils / Liblist.pm
index 2a43022..5388641 100644 (file)
@@ -2,19 +2,20 @@ package ExtUtils::Liblist;
 use vars qw($VERSION);
 # Broken out of MakeMaker from version 4.11
 
-$VERSION = substr q$Revision: 1.2201 $, 10;
+$VERSION = substr q$Revision: 1.25 $, 10;
 
 use Config;
 use Cwd 'cwd';
 use File::Basename;
 
 sub ext {
-  if   ($^O eq 'VMS') { return &_vms_ext;      }
-  else                { return &_unix_os2_ext; }
+  if   ($^O eq 'VMS')     { return &_vms_ext;      }
+  elsif($^O eq 'MSWin32') { return &_win32_ext;    }
+  else                    { return &_unix_os2_ext; }
 }
 
 sub _unix_os2_ext {
-    my($self,$potential_libs, $Verbose) = @_;
+    my($self,$potential_libs, $verbose) = @_;
     if ($^O =~ 'os2' and $Config{libs}) { 
        # Dynamic libraries are not transitive, so we may need including
        # the libraries linked against perl.dll again.
@@ -23,7 +24,7 @@ sub _unix_os2_ext {
        $potential_libs .= $Config{libs};
     }
     return ("", "", "", "") unless $potential_libs;
-    print STDOUT "Potential libraries are '$potential_libs':\n" if $Verbose;
+    warn "Potential libraries are '$potential_libs':\n" if $verbose;
 
     my($so)   = $Config{'so'};
     my($libs) = $Config{'libs'};
@@ -33,7 +34,6 @@ sub _unix_os2_ext {
     # compute $extralibs, $bsloadlibs and $ldloadlibs from
     # $potential_libs
     # this is a rewrite of Andy Dougherty's extliblist in perl
-    # its home is in <distribution>/ext/util
 
     my(@searchpath); # from "-L/path" entries in $potential_libs
     my(@libpath) = split " ", $Config{'libpth'};
@@ -48,12 +48,12 @@ sub _unix_os2_ext {
        if ($thislib =~ s/^(-[LR])//){  # save path flag type
            my($ptype) = $1;
            unless (-d $thislib){
-               print STDOUT "$ptype$thislib ignored, directory does not exist\n"
-                       if $Verbose;
+               warn "$ptype$thislib ignored, directory does not exist\n"
+                       if $verbose;
                next;
            }
            unless ($self->file_name_is_absolute($thislib)) {
-             print STDOUT "Warning: $ptype$thislib changed to $ptype$pwd/$thislib\n";
+             warn "Warning: $ptype$thislib changed to $ptype$pwd/$thislib\n";
              $thislib = $self->catdir($pwd,$thislib);
            }
            push(@searchpath, $thislib);
@@ -64,7 +64,7 @@ sub _unix_os2_ext {
 
        # Handle possible library arguments.
        unless ($thislib =~ s/^-l//){
-         print STDOUT "Unrecognized argument in LIBS ignored: '$thislib'\n";
+         warn "Unrecognized argument in LIBS ignored: '$thislib'\n";
          next;
        }
 
@@ -124,10 +124,10 @@ sub _unix_os2_ext {
                 #
                 # , the compilation tools expand the environment variables.)
            } else {
-               print STDOUT "$thislib not found in $thispth\n" if $Verbose;
+               warn "$thislib not found in $thispth\n" if $verbose;
                next;
            }
-           print STDOUT "'-l$thislib' found at $fullname\n" if $Verbose;
+           warn "'-l$thislib' found at $fullname\n" if $verbose;
            my($fullnamedir) = dirname($fullname);
            push @ld_run_path, $fullnamedir unless $ld_run_path_seen{$fullnamedir}++;
            $found++;
@@ -173,7 +173,7 @@ sub _unix_os2_ext {
            }
            last;       # found one here so don't bother looking further
        }
-       print STDOUT "Note (probably harmless): "
+       warn "Note (probably harmless): "
                     ."No library found for -l$thislib\n"
            unless $found_lib>0;
     }
@@ -181,12 +181,128 @@ sub _unix_os2_ext {
     ("@extralibs", "@bsloadlibs", "@ldloadlibs",join(":",@ld_run_path));
 }
 
+sub _win32_ext {
+
+    require Text::ParseWords;
+
+    my($self, $potential_libs, $verbose) = @_;
+
+    # If user did not supply a list, we punt.
+    # (caller should probably use the list in $Config{libs})
+    return ("", "", "", "") unless $potential_libs;
+
+    my($so)   = $Config{'so'};
+    my($libs) = $Config{'libs'};
+    my($libpth) = $Config{'libpth'};
+    my($libext) = $Config{'lib_ext'} || ".lib";
+
+    if ($libs and $potential_libs !~ /:nodefault/i) { 
+       # If Config.pm defines a set of default libs, we always
+       # tack them on to the user-supplied list, unless the user
+       # specified :nodefault
+
+       $potential_libs .= " " if $potential_libs;
+       $potential_libs .= $libs;
+    }
+    warn "Potential libraries are '$potential_libs':\n" if $verbose;
+
+    # normalize to forward slashes
+    $libpth =~ s,\\,/,g;
+    $potential_libs =~ s,\\,/,g;
+
+    # compute $extralibs from $potential_libs
+
+    my(@searchpath); # from "-L/path" entries in $potential_libs
+    my(@libpath) = Text::ParseWords::quotewords('\s+', 0, $libpth);
+    my(@extralibs);
+    my($fullname, $thislib, $thispth);
+    my($pwd) = cwd(); # from Cwd.pm
+    my($lib) = '';
+    my($found) = 0;
+
+    foreach $thislib (Text::ParseWords::quotewords('\s+', 0, $potential_libs)){
+
+       # Handle possible linker path arguments.
+       if ($thislib =~ s/^-L// and not -d $thislib) {
+           warn "-L$thislib ignored, directory does not exist\n"
+               if $verbose;
+           next;
+       }
+       elsif (-d $thislib) {
+           unless ($self->file_name_is_absolute($thislib)) {
+             warn "Warning: '-L$thislib' changed to '-L$pwd/$thislib'\n";
+             $thislib = $self->catdir($pwd,$thislib);
+           }
+           push(@searchpath, $thislib);
+           next;
+       }
+
+       # Handle possible library arguments.
+       if ($thislib =~ s/^-l// and $thislib !~ /^lib/i) {
+           $thislib = "lib$thislib";
+       }
+       $thislib .= $libext if $thislib !~ /\Q$libext\E$/i;
+
+       my($found_lib)=0;
+       foreach $thispth (@searchpath, @libpath){
+           unless (-f ($fullname="$thispth\\$thislib")) {
+               warn "$thislib not found in $thispth\n" if $verbose;
+               next;
+           }
+           warn "'$thislib' found at $fullname\n" if $verbose;
+           $found++;
+           $found_lib++;
+           push(@extralibs, $fullname);
+           last;
+       }
+       warn "Note (probably harmless): "
+                    ."No library found for '$thislib'\n"
+           unless $found_lib>0;
+    }
+    return ('','','','') unless $found;
+
+    # make sure paths with spaces are properly quoted
+    @extralibs = map { (/\s/ && !/^".*"$/) ? qq["$_"] : $_ } @extralibs;
+    $lib = join(' ',@extralibs);
+    warn "Result: $lib\n" if $verbose;
+    wantarray ? ($lib, '', $lib, '') : $lib;
+}
+
 
 sub _vms_ext {
   my($self, $potential_libs,$verbose) = @_;
-  return ('', '', '', '') unless $potential_libs;
+  my(@crtls,$crtlstr);
+  my($dbgqual) = $self->{OPTIMIZE} || $Config{'optimize'} ||
+                 $self->{CCFLAS}   || $Config{'ccflags'};
+  @crtls = ( ($dbgqual =~ m-/Debug-i ? $Config{'dbgprefix'} : '')
+              . 'PerlShr/Share' );
+  push(@crtls, grep { not /\(/ } split /\s+/, $Config{'libs'});
+  push(@crtls, grep { not /\(/ } split /\s+/, $Config{'libc'});
+  # In general, we pass through the basic libraries from %Config unchanged.
+  # The one exception is that if we're building in the Perl source tree, and
+  # a library spec could be resolved via a logical name, we go to some trouble
+  # to insure that the copy in the local tree is used, rather than one to
+  # which a system-wide logical may point.
+  if ($self->{PERL_SRC}) {
+    my($lib,$locspec,$type);
+    foreach $lib (@crtls) { 
+      if (($locspec,$type) = $lib =~ m-^([\w$\-]+)(/\w+)?- and $locspec =~ /perl/i) {
+        if    (lc $type eq '/share')   { $locspec .= $Config{'exe_ext'}; }
+        elsif (lc $type eq '/library') { $locspec .= $Config{'lib_ext'}; }
+        else                           { $locspec .= $Config{'obj_ext'}; }
+        $locspec = $self->catfile($self->{PERL_SRC},$locspec);
+        $lib = "$locspec$type" if -e $locspec;
+      }
+    }
+  }
+  $crtlstr = @crtls ? join(' ',@crtls) : '';
 
-  my(@dirs,@libs,$dir,$lib,%sh,%olb,%obj);
+  unless ($potential_libs) {
+    warn "Result:\n\tEXTRALIBS: \n\tLDLOADLIBS: $crtlstr\n" if $verbose;
+    return ('', '', $crtlstr, '');
+  }
+
+  my(@dirs,@libs,$dir,$lib,%sh,%olb,%obj,$ldlib);
   my $cwd = cwd();
   my($so,$lib_ext,$obj_ext) = @Config{'so','lib_ext','obj_ext'};
   # List of common Unix library names and there VMS equivalents
@@ -199,7 +315,7 @@ sub _vms_ext {
                  'Xmu' => 'DECW$XMULIBSHR');
   if ($Config{'vms_cc_type'} ne 'decc') { $libmap{'curses'} = 'VAXCCURSE'; }
 
-  print STDOUT "Potential libraries are '$potential_libs'\n" if $verbose;
+  warn "Potential libraries are '$potential_libs'\n" if $verbose;
 
   # First, sort out directories and library names in the input
   foreach $lib (split ' ',$potential_libs) {
@@ -216,11 +332,11 @@ sub _vms_ext {
   # path in a logical name.)
   foreach $dir (@dirs) {
     unless (-d $dir) {
-      print STDOUT "Skipping nonexistent Directory $dir\n" if $verbose > 1;
+      warn "Skipping nonexistent Directory $dir\n" if $verbose > 1;
       $dir = '';
       next;
     }
-    print STDOUT "Resolving directory $dir\n" if $verbose;
+    warn "Resolving directory $dir\n" if $verbose;
     if ($self->file_name_is_absolute($dir)) { $dir = $self->fixpath($dir,1); }
     else                                    { $dir = $self->catdir($cwd,$dir); }
   }
@@ -245,24 +361,24 @@ sub _vms_ext {
       push(@variants,"lib$lib") if $lib !~ /[:>\]]/;
     }
     push(@variants,$lib);
-    print STDOUT "Looking for $lib\n" if $verbose;
+    warn "Looking for $lib\n" if $verbose;
     foreach $variant (@variants) {
       foreach $dir (@dirs) {
         my($type);
 
         $name = "$dir$variant";
-        print "\tChecking $name\n" if $verbose > 2;
+        warn "\tChecking $name\n" if $verbose > 2;
         if (-f ($test = VMS::Filespec::rmsexpand($name))) {
           # It's got its own suffix, so we'll have to figure out the type
           if    ($test =~ /(?:$so|exe)$/i)      { $type = 'sh'; }
           elsif ($test =~ /(?:$lib_ext|olb)$/i) { $type = 'olb'; }
           elsif ($test =~ /(?:$obj_ext|obj)$/i) {
-            print STDOUT "Note (probably harmless): "
+            warn "Note (probably harmless): "
                         ."Plain object file $test found in library list\n";
             $type = 'obj';
           }
           else {
-            print STDOUT "Note (probably harmless): "
+            warn "Note (probably harmless): "
                         ."Unknown library type for $test; assuming shared\n";
             $type = 'sh';
           }
@@ -281,7 +397,7 @@ sub _vms_ext {
         elsif (not length($ctype) and  # If we've got a lib already, don't bother
                ( -f ($test = VMS::Filespec::rmsexpand($name,$obj_ext)) or
                  -f ($test = VMS::Filespec::rmsexpand($name,'.obj'))))  {
-          print STDOUT "Note (probably harmless): "
+          warn "Note (probably harmless): "
                       ."Plain object file $test found in library list\n";
           $type = 'obj';
           $name = $test unless $test =~ /obj;?\d*$/i;
@@ -294,11 +410,11 @@ sub _vms_ext {
       if ($ctype) { 
         eval '$' . $ctype . "{'$cand'}++";
         die "Error recording library: $@" if $@;
-        print STDOUT "\tFound as $cand (really $ctest), type $ctype\n" if $verbose > 1;
+        warn "\tFound as $cand (really $test), type $ctype\n" if $verbose > 1;
         next LIB;
       }
     }
-    print STDOUT "Note (probably harmless): "
+    warn "Note (probably harmless): "
                 ."No library found for $lib\n";
   }
 
@@ -311,8 +427,10 @@ sub _vms_ext {
   push(@libs, map { "$_/Library" } sort keys %olb);
   push(@libs, map { "$_/Share"   } sort keys %sh);
   $lib = join(' ',@libs);
-  print "Result: $lib\n" if $verbose;
-  wantarray ? ($lib, '', $lib, '') : $lib;
+
+  $ldlib = $crtlstr ? "$lib $crtlstr" : $lib;
+  warn "Result:\n\tEXTRALIBS: $lib\n\tLDLOADLIBS: $ldlib\n" if $verbose;
+  wantarray ? ($lib, '', $ldlib, '') : $lib;
 }
 
 1;
@@ -327,7 +445,7 @@ ExtUtils::Liblist - determine libraries to use and how to use them
 
 C<require ExtUtils::Liblist;>
 
-C<ExtUtils::Liblist::ext($potential_libs, $Verbose);>
+C<ExtUtils::Liblist::ext($self, $potential_libs, $verbose);>
 
 =head1 DESCRIPTION
 
@@ -338,7 +456,9 @@ C<-L/another/path> this will affect the searches for all subsequent
 libraries.
 
 It returns an array of four scalar values: EXTRALIBS, BSLOADLIBS,
-LDLOADLIBS, and LD_RUN_PATH.
+LDLOADLIBS, and LD_RUN_PATH.  Some of these don't mean anything
+on VMS and Win32.  See the details about those platform specifics
+below.
 
 Dependent libraries can be linked in one of three ways:
 
@@ -420,8 +540,10 @@ these directives, rather than elements used on the linker command line.
 
 =item *
 
-LDLOADLIBS and EXTRALIBS are always identical under VMS, and BSLOADLIBS
-and LD_RIN_PATH are always empty.
+LDLOADLIBS contains both the libraries found based on C<$potential_libs> and
+the CRTLs, if any, specified in Config.pm.  EXTRALIBS contains just those
+libraries found based on C<$potential_libs>.  BSLOADLIBS and LD_RUN_PATH
+are always empty.
 
 =back
 
@@ -434,6 +556,69 @@ extensions originally designed for a Unix or VMS environment.  If you
 encounter problems, or discover cases where the search could be improved,
 please let us know.
 
+=head2 Win32 implementation
+
+The version of ext() which is executed under Win32 differs from the
+Unix-OS/2 version in several respects:
+
+=over 2
+
+=item *
+
+Input library and path specifications are accepted with or without the
+C<-l> and C<-L> prefices used by Unix linkers.  C<-lfoo> specifies the
+library C<libfoo.lib> (unless C<foo> already starts with C<lib>), and
+C<-Ls:ome\dir> specifies a directory to look for the libraries that follow.
+If neither prefix is present, a token is considered a directory to search
+if it is in fact a directory, and a library to search for otherwise.  The
+C<$Config{lib_ext}> suffix will be appended to any entries that are not
+directories and don't already have the suffix.  Authors who wish their
+extensions to be portable to Unix or OS/2 should use the Unix prefixes,
+since the Unix-OS/2 version of ext() requires them.
+
+=item *
+
+Entries cannot be plain object files, as many Win32 compilers will
+not handle object files in the place of libraries.
+
+=item *
+
+If C<$potential_libs> is empty, the return value will be empty.
+Otherwise, the libraries specified by C<$Config{libs}> (see Config.pm)
+will be appended to the list of C<$potential_libs>.  The libraries
+will be searched for in the directories specified in C<$potential_libs>
+as well as in C<$Config{libpth}>. For each library that is found,  a
+space-separated list of fully qualified library pathnames is generated.
+You may specify an entry that matches C</:nodefault/i> in
+C<$potential_libs> to disable the appending of default libraries
+found in C<$Config{libs}> (this should be only needed very rarely).
+
+=item *
+
+The libraries specified may be a mixture of static libraries and
+import libraries (to link with DLLs).  Since both kinds are used
+pretty transparently on the win32 platform, we do not attempt to
+distinguish between them.
+
+=item *
+
+LDLOADLIBS and EXTRALIBS are always identical under Win32, and BSLOADLIBS
+and LD_RUN_PATH are always empty (this may change in future).
+
+=item *
+
+You must make sure that any paths and path components are properly
+surrounded with double-quotes if they contain spaces. For example,
+C<$potential_libs> could be (literally):
+
+       "-Lc:\Program Files\vc\lib" msvcrt.lib "la test\foo bar.lib"
+
+Note how the first and last entries are protected by quotes in order
+to protect the spaces.
+
+=back
+
+
 =head1 SEE ALSO
 
 L<ExtUtils::MakeMaker>