On Sat, 13 Sep 2025 13:26:22 GMT, Weijun Wang <wei...@openjdk.org> wrote:
> I've seen this before. https://bugs.openjdk.org/browse/JDK-8345578. Thank you Weijun for the review. The conditional code in `Utils` test library class surprised me and looking at why it was introduced, it appears to be done as part of https://bugs.openjdk.org/browse/JDK-8253750. The motivation to do that I think is reasonable, so I won't pursue any changes to the `Utils` class further. For this current PR, would you suggest I wait for further reviews or would it be OK to go ahead with the integration? Locally I have verified this test now works (and used to previously fail). In our CI I have run tier2 and just this test too and that too passed (but that isn't of much consequence because the build number used in those runs will be 0). ------------- PR Comment: https://git.openjdk.org/jdk/pull/27269#issuecomment-3288369632