RFR: 8362533: Tests sun/management/jmxremote/bootstrap/* duplicate VM flags [v2]

Lei Zhu duke at openjdk.org
Thu Jul 31 04:56:54 UTC 2025


On Wed, 30 Jul 2025 22:05:15 GMT, Leonid Mesnik <lmesnik at openjdk.org> 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


More information about the serviceability-dev mailing list