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

Reply via email to