Re: [PATCH] trunk broken for `make check'

[Paolo Bonzini <bonzini at gnu dot org>]

> > The platform is Debian GNU/Linux 3.0, FSF binutils 2.14, gcc3.4.2 used to
> > bootstrap.
> >
> > What have I done wrong?
>
> You need autogen to check fixincludes (fixincludes was just moved to 
> toplevel).

The problem arose when it was made a host module.  Build modules are not 
tested, so this bug is not directly related to moving fixincludes to the 
toplevel: moving fixincludes to the toplevel kept the old behavior of 
only running the fixincludes testsuite on explicit demand.

> Personally I always run 'make -k check' as prescribed in 
> http://gcc.gnu.org/install/test.html, which works around the problem.

Also, not typing "make -k check" will for example cause almost always 
the libjava tests not to run, because boehm-gc's tests exit with a 1 status.

The real fix is to change the toplevel to always behave like make -k, 
see the attached patch.

Paolo

2004-11-18  Paolo Bonzini  <bonzini@gnu.org>

	* Makefile.tpl (check-[+module+], check-target-[+module+],
	check-gcc-c++): Do not fail on errors.
	* Makefile.in: Regenerate.

Index: Makefile.tpl
===================================================================
RCS file: /cvs/gcc/gcc/Makefile.tpl,v
retrieving revision 1.117
diff -u -r1.117 Makefile.tpl
--- Makefile.tpl	13 Nov 2004 07:32:47 -0000	1.117
+++ Makefile.tpl	19 Nov 2004 13:29:14 -0000
@@ -936,8 +936,8 @@
 	  s=`cd $(srcdir); ${PWD_COMMAND}`; export s; \
 	  $(SET_LIB_PATH) \
 	  $(HOST_EXPORTS) \
-	  (cd [+module+] && $(MAKE) $(FLAGS_TO_PASS) [+ 
-	    extra_make_flags+] check); \
+	  (set +e; cd [+module+] && $(MAKE) -k $(FLAGS_TO_PASS) [+ 
+	    extra_make_flags+] check; exit 0); \
 	fi
 [+ ELSE check +]
 check-[+module+]:
@@ -945,8 +945,8 @@
 	s=`cd $(srcdir); ${PWD_COMMAND}`; export s; \
 	$(SET_LIB_PATH) \
 	$(HOST_EXPORTS) \
-	(cd [+module+] && $(MAKE) $(FLAGS_TO_PASS)[+ 
-	  extra_make_flags+] check)
+	(set +e; cd [+module+] && $(MAKE) -k $(FLAGS_TO_PASS)[+ 
+	  extra_make_flags+] check; exit 0)
 [+ ENDIF no_check +]
 @endif [+module+]
 
@@ -1109,12 +1109,12 @@
 ELSE normal_cxx +]
 	$(NORMAL_TARGET_EXPORTS) \[+
 ENDIF raw_cxx +]
-	(cd $(TARGET_SUBDIR)/[+module+] && \
-	  $(MAKE) $(TARGET_FLAGS_TO_PASS) [+
+	(set +e; cd $(TARGET_SUBDIR)/[+module+] && \
+	  $(MAKE) -k $(TARGET_FLAGS_TO_PASS) [+
 	    IF raw_cxx 
 	      +] 'CXX=$$(RAW_CXX_FOR_TARGET)' 'CXX_FOR_TARGET=$$(RAW_CXX_FOR_TARGET)' [+ 
 	    ENDIF raw_cxx 
-	  +] [+extra_make_flags+] check)
+	  +] [+extra_make_flags+] check; exit 0)
 [+ ENDIF no_check +]
 @endif target-[+module+]
 
@@ -1265,7 +1265,8 @@
 	  s=`cd $(srcdir); ${PWD_COMMAND}`; export s; \
 	  $(SET_LIB_PATH) \
 	  $(HOST_EXPORTS) \
-	  (cd gcc && $(MAKE) $(GCC_FLAGS_TO_PASS) check-c++); \
+	  (set +e; cd gcc && $(MAKE) -k $(GCC_FLAGS_TO_PASS) check-c++; \
+	    exit 0); \
 	else \
 	  true; \
 	fi