Change looks good to me.
/Erik
On 2013-06-14 01:53, Mike Duigou wrote:
Hello all;
This is a small change to the make test target with big consequences. It
enables concurrent running of tests. The number of simultaneous test jobs is
the same as number of build jobs.
This issue probably should have been combined with 8014819 as the two issues
have been tested together. Thanks go to Amy Lu and Alan Bateman for their
feedback and experience reports with concurrent testing.
JPRT is not impacted by this change since it invokes the test/Makefile directly.
Mike
diff -r b40d24f793d2 common/makefiles/Main.gmk
--- a/common/makefiles/Main.gmk Tue Jun 04 10:36:00 2013 +0200
+++ b/common/makefiles/Main.gmk Thu Jun 13 16:47:45 2013 -0700
@@ -183,7 +183,7 @@
test: images test-only
test-only: start-make
@$(call TargetEnter)
- @($(CD) $(SRC_ROOT)/test&& $(BUILD_LOG_WRAPPER) $(MAKE) -j1 -k
JT_HOME=$(JT_HOME) MAKEFLAGS= PRODUCT_HOME=$(JDK_IMAGE_DIR)
JPRT_JAVA_HOME=$(JDK_IMAGE_DIR) ALT_OUTPUTDIR=$(OUTPUT_ROOT) $(TEST)) || true
+ @($(CD) $(SRC_ROOT)/test&& $(BUILD_LOG_WRAPPER) $(MAKE) -j1 -k
MAKEFLAGS= JT_HOME=$(JT_HOME) PRODUCT_HOME=$(JDK_IMAGE_DIR)
JPRT_JAVA_HOME=$(JDK_IMAGE_DIR) ALT_OUTPUTDIR=$(OUTPUT_ROOT) CONCURRENCY=$(JOBS)
$(TEST)) || true
@$(call TargetExit)
# Stores the tips for each repository. This file is be used when constructing
the jdk image and can be
