From 59dd3af6a2407e6ed3cfcb9b3b6f711cc7aba0ec Mon Sep 17 00:00:00 2001 From: Jan Kratochvil Date: Tue, 30 Jun 2009 21:23:38 +0000 Subject: [PATCH] gdb/ * Makefile.in (TARGET_FLAGS_TO_PASS): Include also FORCE_PARALLEL. (check//%): Construct FORCE_PARALLEL for the sub-make. Extend the comment for it. gdb/testsuite/ Fix `make check//%' target after `make check' has been ran. * Makefile.in (TEST_TARGETS): Remove the %/.dir dependency. Add a mkdir call. (check-gdb.base%): Remove the gdb.base%/.dir dependency. (%/.dir): Remove. --- gdb/ChangeLog | 6 ++++++ gdb/Makefile.in | 7 +++++-- gdb/testsuite/ChangeLog | 8 ++++++++ gdb/testsuite/Makefile.in | 9 +++------ 4 files changed, 22 insertions(+), 8 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index e95f19478ac..41c9930f51b 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2009-06-30 Jan Kratochvil + + * Makefile.in (TARGET_FLAGS_TO_PASS): Include also FORCE_PARALLEL. + (check//%): Construct FORCE_PARALLEL for the sub-make. Extend the + comment for it. + 2009-06-30 Paul Pluzhnikov gdb/10275 diff --git a/gdb/Makefile.in b/gdb/Makefile.in index a007949fce0..3852e064653 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -620,7 +620,8 @@ TARGET_FLAGS_TO_PASS = \ "MAKEINFO=$(MAKEINFO)" \ "MAKEHTML=$(MAKEHTML)" \ "RUNTEST=$(RUNTEST)" \ - "RUNTESTFLAGS=$(RUNTESTFLAGS)" + "RUNTESTFLAGS=$(RUNTESTFLAGS)" \ + "FORCE_PARALLEL=$(FORCE_PARALLEL)" # All source files that go into linking GDB. # Links made at configuration time should not be specified here, since @@ -885,7 +886,8 @@ check: force # will run 3 concurrent sessions of check, eventually testing all 10 # combinations. GNU make is required for the % pattern to work, as is # a shell that expands alternations within braces. If GNU make is not -# used, this rule will harmlessly fail to match. +# used, this rule will harmlessly fail to match. Used FORCE_PARALLEL to +# prevent serialized checking due to the passed RUNTESTFLAGS. check//%: force @if [ -f testsuite/config.status ]; then \ rootme=`pwd`; export rootme; \ @@ -905,6 +907,7 @@ check//%: force else :; fi && cd $$testdir && \ $(MAKE) $(TARGET_FLAGS_TO_PASS) \ RUNTESTFLAGS="--target_board=$$variant $(RUNTESTFLAGS)" \ + FORCE_PARALLEL=$(if $(FORCE_PARALLEL),1,$(if $(RUNTESTFLAGS),,1)) \ "$$target"; \ else true; fi diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 54a97dc0977..1897d89ef5a 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,11 @@ +2009-06-30 Jan Kratochvil + + Fix `make check//%' target after `make check' has been ran. + * Makefile.in (TEST_TARGETS): Remove the %/.dir dependency. Add + a mkdir call. + (check-gdb.base%): Remove the gdb.base%/.dir dependency. + (%/.dir): Remove. + 2009-06-30 Jan Kratochvil * lib/mi-support.exp (mi_check_thread_states): Permit any output before diff --git a/gdb/testsuite/Makefile.in b/gdb/testsuite/Makefile.in index 5344418664c..c574e230004 100644 --- a/gdb/testsuite/Makefile.in +++ b/gdb/testsuite/Makefile.in @@ -170,7 +170,8 @@ check-parallel: $(SHELL) $(srcdir)/dg-extract-results.sh -L \ $(addsuffix /gdb.log,$(TEST_DIRS)) > gdb.log -$(filter-out check-gdb.base%,$(TEST_TARGETS)): check-gdb.%: all site.exp gdb.%/.dir +$(filter-out check-gdb.base%,$(TEST_TARGETS)): check-gdb.%: all site.exp + @if test ! -d gdb.$*; then mkdir gdb.$*; fi $(DO_RUNTEST) --directory=gdb.$* --outdir=gdb.$* $(RUNTESTFLAGS) # Each half (roughly) of the .exp files from gdb.base. @@ -178,14 +179,10 @@ BASE1_FILES = $(patsubst $(srcdir)/%,%,$(wildcard $(srcdir)/gdb.base/[a-m]*.exp) BASE2_FILES = $(patsubst $(srcdir)/%,%,$(wildcard $(srcdir)/gdb.base/[n-z]*.exp)) # Handle each half of gdb.base. -check-gdb.base%: all site.exp gdb.base%/.dir +check-gdb.base%: all site.exp @if test ! -d gdb.base$*; then mkdir gdb.base$*; fi $(DO_RUNTEST) $(BASE$*_FILES) --outdir gdb.base$* $(RUNTESTFLAGS) -%/.dir: - @-if test ! -d $*; then mkdir $*; fi - @echo > $@ - subdir_do: force @for i in $(DODIRS); do \ if [ -d ./$$i ] ; then \ -- 2.30.2