[Bug bootstrap/37923] [4.4 Regression] CPPFLAGS now unset for stage 1 build of libcpp files.

["howarth at nitro dot med dot uc dot edu" <gcc-bugzilla at gcc dot gnu dot org>]

------- Comment #15 from howarth at nitro dot med dot uc dot edu  2008-10-27 13:39 -------
Looking at diffs (attached) between the Makefiles in the toplevel and libcpp
subdirectory before and after drow's changes, I don't see any significant
differences in the libcpp level Makefile. However I suspect the problem with
the CPPFLAGS becoming unset after drow's patch is due to the instance of...

@@ -343,6 +347,7 @@
 GNATMAKE = no

 CFLAGS = -g -O2
+CPPFLAGS =
 LDFLAGS =
 LIBCFLAGS = $(CFLAGS)
 CXXFLAGS = -g -O2

which comes after...

@@ -147,9 +148,10 @@
        WINDMC="$(WINDMC_FOR_BUILD)"; export WINDMC;

 # These variables must be set on the make command line for directories
-# built for the build system to override those in BASE_FLAGS_TO_PASSS.
+# built for the build system to override those in BASE_FLAGS_TO_PASS.
 EXTRA_BUILD_FLAGS = \
        CFLAGS="$(CFLAGS_FOR_BUILD)" \
+       CPPFLAGS="$(CPPFLAGS_FOR_BUILD)" \
        LDFLAGS="$(LDFLAGS_FOR_BUILD)"

 # This is the list of directories to built for the host system.
@@ -169,6 +171,7 @@
        ADA_CFLAGS="$(ADA_CFLAGS)"; export ADA_CFLAGS; \
        CFLAGS="$(CFLAGS)"; export CFLAGS; \
        CONFIG_SHELL="$(SHELL)"; export CONFIG_SHELL; \
+       CPPFLAGS="$(CPPFLAGS)"; export CPPFLAGS; \
        CXX="$(CXX)"; export CXX; \
        CXXFLAGS="$(CXXFLAGS)"; export CXXFLAGS; \
        AR="$(AR)"; export AR; \

Wouldn't setting CPPFLAGS= to a null string effectively undo the previous
exports of CPPFLAGS?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37923