On Wed, 30 Jul 2025 22:05:15 GMT, Leonid Mesnik <[email protected]> wrote:
> Comment generic to all three test fixes: Shouldn't be
> command.add(TEST_CLASS_PATH); command.add(className); also removed? The
> ProcessTools.createTestJavaProcessBuilder(command); uses standard classpath
> that should fit this test needs. If needed, property "test.noclasspath",
> mgith be use to don't add classpath. See
> jdk/test/lib/process/ProcessTools.java:161 private static ProcessBuilder
> createJavaProcessBuilder(String... command) { ... String noCPString =
> System.getProperty("test.noclasspath", "false");
Thanks for review, the duplicate `-cp` has been removed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26555#discussion_r2244348086
