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