On Fri, 28 Feb 2025 01:40:34 GMT, Jiangli Zhou <jian...@openjdk.org> wrote:
> Please review the test fix that removes `libVThreadEventTest` explicit > dependency to `libjvm`, by removing the call to `JNI_GetCreatedJavaVMs` in > `Agent_OnAttach`. There is a `vm` argument passed via `Agent_OnAttach`. With > the change, `libVThreadEventTest` no longer needs to be linked with > `libjvm.so` (or `jvm.dll`). `VThreadEventTest.java` now runs on both dynamic > and static JDK. > > Tested on static JDK locally > GHA: > https://github.com/jianglizhou/jdk/actions/runs/13577993687/job/37960245161 Okay, great! I did not notice it was for exe's, I just grepped quickly for `libjvm`. ------------- PR Comment: https://git.openjdk.org/jdk/pull/23835#issuecomment-2710224648
