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

"Sisyphus" <[email protected]>
Newsgroups gmane.comp.lang.perl.modules.extutils-makemaker.devel
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 = $_;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.