RFR: 8319673: Few security tests ignore VM flags [v5]

Hai-May Chao hchao at openjdk.org
Mon Feb 26 18:07:43 UTC 2024


On Thu, 22 Feb 2024 12:52:19 GMT, Matthew Donovan <mdonovan at openjdk.org> wrote:

>> In this PR I updated the tests to use the newer ProcessTools.createTestJavaProcessBuilder() methods to launch child processes. This method ensures that VM options are passed to subprocesses.
>
> Matthew Donovan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   reverted Compatibility.java to avoid possible conflicts with VM options and older JDKs.

Nit: Copyright year of IterationCount.java would need to be changed.

Looks good. Thanks.

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

Marked as reviewed by hchao (Committer).

PR Review: https://git.openjdk.org/jdk/pull/17877#pullrequestreview-1901649894
PR Comment: https://git.openjdk.org/jdk/pull/17877#issuecomment-1964804507



More information about the security-dev mailing list