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

Reply via email to