8157605: Can't set both CONCURRENCY and EXTRA_JTREG_OPTIONS when running tests
authorsla
Tue, 24 May 2016 12:15:23 +0200
changeset 38502 93e34c3629a7
parent 38501 42bbe0b77321
child 38503 6e1d6bbe9cb9
8157605: Can't set both CONCURRENCY and EXTRA_JTREG_OPTIONS when running tests Reviewed-by: tbell, erikj
jaxp/test/Makefile
--- a/jaxp/test/Makefile	Sat May 21 08:00:14 2016 +0100
+++ b/jaxp/test/Makefile	Tue May 24 12:15:23 2016 +0200
@@ -265,7 +265,10 @@
 # ------------------------------------------------------------------
 
 ifdef CONCURRENCY
-  EXTRA_JTREG_OPTIONS += -concurrency:$(CONCURRENCY)
+  JTREG_BASIC_OPTIONS += -concurrency:$(CONCURRENCY)
+endif
+ifdef EXTRA_JTREG_OPTIONS
+  JTREG_BASIC_OPTIONS += $(EXTRA_JTREG_OPTIONS)
 endif
 
 # Default JTREG to run
@@ -290,8 +293,6 @@
 # Set the max memory for jtreg control vm
 JTREG_MEMORY_OPTION = -J-Xmx512m
 JTREG_BASIC_OPTIONS += $(JTREG_MEMORY_OPTION)
-# Add any extra options
-JTREG_BASIC_OPTIONS += $(EXTRA_JTREG_OPTIONS)
 # Set other vm and test options
 JTREG_TEST_OPTIONS = $(JAVA_ARGS:%=-javaoptions:%) $(JAVA_VM_ARGS:%=-vmoption:%)
 # Set the GC options for test vms