[Win32] Feature request: acknowledge $ENV{LIBRARY_PATH}

[email protected] ("Sisyphus") Sun, 26 Apr 2009 17:48:54 +1000
Newsgroups perl.makemaker
Message-ID <C765C7823DA64015911A025B0AFE03DF@desktop2>
Hi,

Could the attached patch (against version 6.50 of ExtUtils/Liblist/Kid.pm) 
be applied ?

It adds extra directories to $Config{libpth}, and thus enables additional 
libraries to be found without having to specify their locations with the 
'-L' switch (iff those locations have been specified in $ENV{LIBRARY_PATH}).
gcc (in general, not *just* on windows) already searches the locations 
specified in $ENV{LIBRARY_PATH} - we just need to enable MakeMaker to find 
them, and this patch does that for Windows.

We could even port this feature to other operating systems - though we'd 
have to split on $Config{path_sep} rather than ';' and the code would have 
to be moved out of sub _win32_ext. However, I doubt that there's much call 
for this on other systems.
In fact, there's probably not even much of a demand for this on Windows - 
but it would certainly be a useful feature for me.

Cheers,
Rob
diff.txt (text/plain, 506 B)
--- C:/_32/comp/Extutils-Makemaker-6.50/lib/Extutils/Liblist/Kid.pm	Sun Apr 26 09:12:00 2009
+++ C:/perl510_M/5.10.0/lib/ExtUtils/Liblist/Kid.pm	Sun Apr 26 09:21:40 2009
@@ -267,6 +267,11 @@
         push @libpath, split /;/, $ENV{LIB};
     }
 
+    # For the benefit of MinGW:
+    if ($GC and exists $ENV{LIBRARY_PATH} and $ENV{LIBRARY_PATH}) {
+        push @libpath, split /;/, $ENV{LIBRARY_PATH};
+    }
+
     foreach (Text::ParseWords::quotewords('\s+', 0, $potential_libs)){
 
 	my $thislib = $_;