------- Comment #20 from bonzini at gnu dot org 2008-10-27 14:49 -------
I checked config and I see:
- acinclude.m4 CYG_AC_PATH_SIM sets CPPFLAGS, not a problem in this case
- tcl.m4 is not a problem like acinclude.m4 is not
- gettext.m4 saves CPPFLAGS and restores it
- iconv.m4 does the same
- lib-link.m4 likes to modify CPPFLAGS (see AC_LIB_APPENDTOVAR) but only in
AC_LIB_LINKFLAGS which is not used by the previous two files
- lib-prefix.m4 is similarly not a problem
The libcppmakefile.diff attachment does not show a change in CPPFLAGS indeed,
but it shows that the first file is stage>1, and the second file is stage1.
Was this meant? Use "make stage1-start" on both trees" before diffing.
Anyway, the desired -I/sw/include flag should be in the two lines of
libcpp/Makefile corresponding to
INCINTL = @INCINTL@
INCLUDES = -I$(srcdir) -I. -I$(srcdir)/../include @INCINTL@ \
-I$(srcdir)/include
rather than in CPPFLAGS, and it should be picked up from $(INCLUDES) *twice*,
via
ALL_CFLAGS = $(CFLAGS) $(WARN_CFLAGS) $(INCLUDES) $(CPPFLAGS)
COMPILE.base = $(CC) $(DEFS) $(INCLUDES) $(CPPFLAGS) $(ALL_CFLAGS) -c
I think the bug should be fixed by
Index: intl/configure.ac
===================================================================
--- intl/configure.ac (revision 141134)
+++ intl/configure.ac (working copy)
@@ -37,7 +37,6 @@ AC_SUBST(LIBINTL_DEP)
AC_SUBST(INCINTL)
LIBINTL_DEP=
-INCINTL=
case $USE_INCLUDED_LIBINTL in
yes)
LIBINTL=`echo $LIBINTL | sed 's,${top_builddir},&/..,' `
since the invocation of AM_GNU_GETTEXT earlier in intl/configure.ac should have
set INCINTL to the correct value for --without-included-gettext. If this is
confirmed, the bug has been present since the introduction of the current
scheme for gettext configury.
r69071 | zack | 2003-07-08 07:55:11 +0200 (Tue, 08 Jul 2003) | 53 lines
Jack, can you check the above small patch?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37923
