Looks good.

/Erik


On 2017-11-27 15:11, Magnus Ihse Bursie wrote:
From the bug report:

A common scenario is adding vm arguments when running tests. While this can be accomplished by JTREG="VM_OPTIONS=-Xfoo", this is a lot to type for a common case. In the future, when gtest accepts vm arguments as well, a separate, similar but yet different GTEST="VM_OPTIONS=-Xfoo" would be needed.

A new "global" TEST_VM_OPTS=X should behave as if VM_OPTIONS=X were present on all applicable test framework option control variables.

My solution also adds a new "global" TEST_OPTS, where TEST_VM_OPTS=-Xfoo behaves identical to TEST_OPTS="VM_OPTIONS=-Xfoo", but which also allows other global settings. It also adds support for VM_OPTIONS to gtest, and it improves error resilience in gtest result parsing.

Bug: https://bugs.openjdk.java.net/browse/JDK-8177956
WebRev: http://cr.openjdk.java.net/~ihse/JDK-8177956-add-TEST_OPTS/webrev.01

/Magnus


Reply via email to