gdb/
authorJan Kratochvil <jan.kratochvil@redhat.com>
Tue, 30 Jun 2009 21:23:38 +0000 (21:23 +0000)
committerJan Kratochvil <jan.kratochvil@redhat.com>
Tue, 30 Jun 2009 21:23:38 +0000 (21:23 +0000)
* 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
gdb/Makefile.in
gdb/testsuite/ChangeLog
gdb/testsuite/Makefile.in

index e95f19478acb899c60b469fc3b12df23188cfd7d..41c9930f51b6a863a41b78105494f70f034dbf1f 100644 (file)
@@ -1,3 +1,9 @@
+2009-06-30  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       * 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  <ppluzhnikov@google.com>
 
        gdb/10275
index a007949fce0c872717497231d5b963b1c82bae2f..3852e0646530b3ec1a23a83583ba55c762ef075d 100644 (file)
@@ -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
 
index 54a97dc0977836738117870edc4769fbf3c4a39d..1897d89ef5a1e86e099b39c40bc101a3319fe78e 100644 (file)
@@ -1,3 +1,11 @@
+2009-06-30  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       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  <jan.kratochvil@redhat.com>
 
        * lib/mi-support.exp (mi_check_thread_states): Permit any output before
index 5344418664c1dfdcc01d0cf7ad18dc93b28cdac8..c574e2300046dba69f9560eb789114d7e07116df 100644 (file)
@@ -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 \