On Thu, 21 Nov 2024 15:20:23 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

> I am not sure this is necessary, the test already imports several other 
> classes from that library (import jdk.test.lib.RandomFactory; import 
> jdk.test.lib.net.SimpleSSLContext;) and not all are listed in the@build.

It's likely it's not needed in this test, but for `jdk.test.lib.Utils` I keep 
an extra eye because it has shown to cause intermittent issues at least in a 
couple of occasions:

https://github.com/openjdk/jdk/pull/20228
https://github.com/openjdk/jdk/pull/20373#issuecomment-2268375366

Thank you for the update. This looks good to me.

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

PR Comment: https://git.openjdk.org/jdk/pull/22278#issuecomment-2492613554

Reply via email to