On Fri, 6 Oct 2023 22:34:40 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:
>> Leonid Mesnik has updated the pull request incrementally with one additional >> commit since the last revision: >> >> simplifed usage, removed classpath adding. > > test/jdk/sun/management/jmxremote/bootstrap/CustomLauncherTest.java line 100: > >> 98: ProcessBuilder client = >> ProcessTools.createJavaProcessBuilder( >> 99: "-cp", >> 100: Utils.TEST_CLASS_PATH, > > Just a question: Is the `-cp` options added by the > `ProcessTools.createTestJvm()`? It is added even by ProcessTools.createJavaProcessBuilder(..) so it is not needed here. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16048#discussion_r1349386018
