On Mon, 20 Nov 2023 19:15:18 GMT, Justin Lu <j...@openjdk.org> wrote:

>> Please review this PR which allows these _j.util_ tests to launch new JVM 
>> processes with VM flags,
>> 
>> This is primarily done using by switching to 
>> `ProcessTools::createTestJavaProcessBuilder`.
>
> Justin Lu has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Pass test.java.opts to PropertiesTest.sh

LGTM. It is out of this fix's scope, but `PropertiesTest.sh` may better be 
converted into a Java test case.

-------------

Marked as reviewed by naoto (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16705#pullrequestreview-1740822747

Reply via email to