RFR: JDK-8174895: test/TestCommon.gmk: value of JTREG_TESTVM_MEMORY_OPTION is missing1/2acghpy

[Magnus Ihse Bursie]

On 2017-02-14 09:42, Erik Joelsson wrote:
> 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:
>
Looks good to me.

/Magnus

> 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
>