On Sun, 8 Feb 2004 11:03:01 +1100, [EMAIL PROTECTED] (Ron Savage) wrote:

>On Sat, 07 Feb 2004 14:31:23 +0000, Andy Hassall wrote:

 [ sorry about the uuencoded split message; recently moved to the NNTP server
instead of email, and I hadn't properly reviewed the attachment settings for my
newsreader ]

>In the interim, however, it has been pointed out that it's not so much the dirs we've 
>mentioned that are relevant, but rather the INCLUDE and LIB dirs.
>
>What say you rework to patch to refer to them?

 OK... attached checks both LIB and INCLUDE are set, and if not prints a
slightly reworded warning.

-- 
Andy Hassall <[EMAIL PROTECTED]> / Space: disk usage analysis tool
<http://www.andyh.co.uk> / <http://www.andyhsoftware.co.uk/space>
--- Makefile.PL.orig    2004-02-08 23:09:49.887336000 +0000
+++ Makefile.PL 2004-02-08 23:09:34.224814400 +0000
@@ -109,6 +109,23 @@
    \tEspecially if you have any problems.\n
 " unless $::opt_s;
 
+if (   $os eq 'MSWin32'
+    && $Config{cc} eq 'cl'
+    && !(exists $ENV{'LIB'} && exists $ENV{'INCLUDE'}))
+{
+    print <<EOT;
+
+*** Using Microsoft Visual C++ compiler, but the LIB and INCLUDE environment
+    variables are not both set. If you have 'unresolved external symbol'
+    errors during linking, run the vcvars32.bat batch file to set up your
+    LIB and INCLUDE environment variables, and try again.
+    A copy of vcvars32.bat can typically be found in the following
+    directories under your Visual Studio install directory:
+    Visual C++ 6.0:     vc98\\bin
+    Visual Studio .NET: vc7\\bin
+
+EOT
+}
 
 # --- Where is Oracle installed...
 
@@ -1494,5 +1511,3 @@
 }
 
 __END__
-
-

Reply via email to