RFR: JDK-8174895: test/TestCommon.gmk: value of JTREG_TESTVM_MEMORY_OPTION is missing1/2acghpy
Erik Joelsson
erik.joelsson at oracle.com
Tue Feb 14 08:42:07 UTC 2017
There is a small bug in the new test/TestCommon.gmk that prevents the
JTREG_TESTVM_MEMORY_OPTION from propagating to the jtreg command line.
The fix is to append to JTREG_TEST_OPTIONS instead of overwriting the
current value further down in the file. Patch provided by Amy Lu.
Bug: https://bugs.openjdk.java.net/browse/JDK-8174895
Patch:
diff -r 4eb77fb98952 test/TestCommon.gmk
--- a/test/TestCommon.gmk
+++ b/test/TestCommon.gmk
@@ -370,7 +370,7 @@
# Give tests access to JT_JAVA, see JDK-8141609
JTREG_BASIC_OPTIONS += -e:JDK8_HOME=${JT_JAVA}
# Set other vm and test options
-JTREG_TEST_OPTIONS = $(JAVA_ARGS:%=-javaoptions:%)
$(JAVA_OPTIONS:%=-vmoption:%) $(JAVA_VM_ARGS:%=-vmoption:%)
+JTREG_TEST_OPTIONS += $(JAVA_ARGS:%=-javaoptions:%)
$(JAVA_OPTIONS:%=-vmoption:%) $(JAVA_VM_ARGS:%=-vmoption:%)
ifeq ($(IGNORE_MARKED_TESTS), true)
# Option to tell jtreg to not run tests marked with "ignore"
/Erik
More information about the build-dev
mailing list