Looks fine,
Thanks for the test update.
Roger
On 12/09/2018 02:59 AM, Philip Race wrote:
Here is an update with a Java-only test, using ProcessBuilder.
http://cr.openjdk.java.net/~prr/8212703.1/index.html
-phil.
On 12/1/18, 9:27 AM, Alan Bateman wrote:
On 30/11/2018 23:11, Roger Riggs wrote:
Hi Phil,
That looks fine.
Too bad it introduces a new shell test, we're trying to get rid of
them.
The other alternative would have the test program needed to set the
environment
and launch a java program.
The implementation look good to me too and I agree with Roger that we
shouldn't be introducing a shell test for this. In this case, it
looks like you just need a /othervm test that checks for the env
variable and property.
-Alan
