On Mon, 7 Oct 2024 16:17:59 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

>> Remove the support for on-demand debugging via the onjcmd option.
>
> test/jdk/jdk/jfr/event/runtime/TestAgentEvent.java line 68:
> 
>> 66:  *      testJavaDynamic
>> 67:  *
>> 68:  * @run main/othervm -XX:+UnlockExperimentalVMOptions 
>> -XX:-UseFastUnorderedTimeStamps 
>> -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=:5005
> 
> This use of port 5005 could possibly conflict with other tests running in 
> parallel that were already dynamically assigned this port. Is there a reason 
> it can't remain "any"?

Good catch, I'll try using `any` tomorrow.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21387#discussion_r1790719642

Reply via email to