Fix make.h preprocessor directive to work better with Windows compilers.
authorPaul Smith <psmith@gnu.org>
Mon, 29 Aug 2005 14:11:00 +0000 (14:11 +0000)
committerPaul Smith <psmith@gnu.org>
Mon, 29 Aug 2005 14:11:00 +0000 (14:11 +0000)
Fix some regression tests to (hopefully) work better on Windows.

ChangeLog
function.c
make.h
tests/ChangeLog
tests/scripts/features/patspecific_vars
tests/scripts/functions/abspath
tests/scripts/options/dash-I

index 067d8fc..d2e47e9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,9 +1,20 @@
+2005-08-29  Paul D. Smith  <psmith@gnu.org>
+
+       * function.c (func_error): On Windows, output from $(info ...)
+       seems to come in the wrong order.  Try to force it with fflush().
+
 2005-08-10  Boris Kolpackov  <boris@kolpackov.net>
 
        * read.c (record_files): Move code that sets stem for static
        pattern rules out of the if (!two_colon) condition so it is
        also executed for two-colon rules. Fixes Savannah bug #13881.
 
+2005-08-08  Paul D. Smith  <psmith@gnu.org>
+
+       * make.h: Don't test that __STDC__ is non-0.  Some compilers
+       (Windows for example) set it to 0 to denote "ISO C + extensions".
+       Fixes bug # 13594.
+
 2005-08-07  Paul D. Smith  <psmith@gnu.org>
 
        * w32/pathstuff.c (getcwd_fs): Fix warning about assignment in a
index 5bf6019..47cdafa 100644 (file)
@@ -1096,6 +1096,7 @@ func_error (char *o, char **argv, const char *funcname)
 
     case 'i':
       printf ("%s\n", msg);
+      fflush(stdout);
       break;
 
     default:
diff --git a/make.h b/make.h
index b772d77..8636928 100644 (file)
--- a/make.h
+++ b/make.h
@@ -42,7 +42,7 @@ char *alloca ();
 
 
 /* Use prototypes if available.  */
-#if defined (__cplusplus) || (defined (__STDC__) && __STDC__)
+#if defined (__cplusplus) || defined (__STDC__)
 # undef  PARAMS
 # define PARAMS(protos)  protos
 #else /* Not C++ or ANSI C.  */
index 84f40f6..5e1f948 100644 (file)
@@ -1,3 +1,13 @@
+2005-08-29  Paul D. Smith  <psmith@gnu.org>
+
+       * scripts/functions/abspath: Add some text to the error messages
+       to get a better idea of what's wrong.  Make warnings instead of
+       errors.
+
+       * scripts/features/patspecific_vars: Don't use "test", which is
+       UNIX specific.  Print the values and let the test script match
+       them.
+
 2005-08-25  Paul Smith  <psmith@gnu.org>
 
        * scripts/variables/SHELL: Use a /./ prefix instead of //: the
index 9e98b43..20c1cfc 100644 (file)
@@ -67,8 +67,8 @@ run_make_test('
 /%: export foo := foo
 
 /bar:
-       @test "$(foo)" = "$$foo"
-', '', '');
+       @echo $(foo) $$foo
+', '', 'foo foo');
 
 
 # TEST #6 -- test expansion of pattern-specific simple variables
index d419255..84c30ab 100644 (file)
@@ -5,68 +5,68 @@ $details = "";
 
 run_make_test('
 ifneq ($(realpath $(abspath .)),$(CURDIR))
-  $(error )
+  $(warning .: abs="$(abspath .)" real="$(realpath $(abspath .))" curdir="$(CURDIR)")
 endif
 
 ifneq ($(realpath $(abspath ./)),$(CURDIR))
-  $(error )
+  $(warning ./: abs="$(abspath ./)" real="$(realpath $(abspath ./))" curdir="$(CURDIR)")
 endif
 
 ifneq ($(realpath $(abspath .///)),$(CURDIR))
-  $(error )
+  $(warning .///: abs="$(abspath .///)" real="$(realpath $(abspath .///))" curdir="$(CURDIR)")
 endif
 
 ifneq ($(abspath /),/)
-  $(error )
+  $(warning /: abspath="$(abspath /)")
 endif
 
 ifneq ($(abspath ///),/)
-  $(error )
+  $(warning ///: abspath="$(abspath ///)")
 endif
 
 ifneq ($(abspath /.),/)
-  $(error )
+  $(warning /.: abspath="$(abspath /.)")
 endif
 
 ifneq ($(abspath ///.),/)
-  $(error )
+  $(warning ///.: abspath="$(abspath ///.)")
 endif
 
 ifneq ($(abspath /./),/)
-  $(error )
+  $(warning /./: abspath="$(abspath /./)")
 endif
 
 ifneq ($(abspath /.///),/)
-  $(error )
+  $(warning /.///: abspath="$(abspath /.///)")
 endif
 
 ifneq ($(abspath /..),/)
-  $(error )
+  $(warning /..: abspath="$(abspath /..)")
 endif
 
 ifneq ($(abspath ///..),/)
-  $(error )
+  $(warning ///..: abspath="$(abspath ///..)")
 endif
 
 ifneq ($(abspath /../),/)
-  $(error )
+  $(warning /../: abspath="$(abspath /../)")
 endif
 
 ifneq ($(abspath /..///),/)
-  $(error )
+  $(warning /..///: abspath="$(abspath /..///)")
 endif
 
 
 ifneq ($(abspath /foo/bar/..),/foo)
-  $(error )
+  $(warning /foo/bar/..: abspath="$(abspath /foo/bar/..)")
 endif
 
 ifneq ($(abspath /foo/bar/../../../baz),/baz)
-  $(error )
+  $(warning /foo/bar/../../../baz: abspath="$(abspath /foo/bar/../../../baz)")
 endif
 
 ifneq ($(abspath /foo/bar/../ /..),/foo /)
-  $(error )
+  $(warning /foo/bar/../ /..: abspath="$(abspath /foo/bar/../ /..)")
 endif
 
 
index 0be0bd7..8dc5d9b 100644 (file)
@@ -1,3 +1,5 @@
+#                                                                    -*-perl-*-
+
 $description ="The following test creates a makefile to test the -I option.";
 
 $details = "\