Skip to content

Conversation

@adamfarley
Copy link
Contributor

@adamfarley adamfarley commented Jul 25, 2025

Related: #6430

Signed-off-by: Adam Farley <adfarley@ibm.com>
@adamfarley
Copy link
Contributor Author

Requesting reviews from @smlambert and @sophia-guo please.

@adamfarley
Copy link
Contributor Author

Requesting review from @llxia as well, please.

@smlambert
Copy link
Contributor

smlambert commented Jul 25, 2025

thanks @adamfarley - I have approved it as it partially addresses #6430, to fully close it, you could also deal with the duplicate entry of sun/security/ssl/X509TrustManagerImpl/distrust/Symantec.java mentioned in that issue as well. Ignore, early morning review (I misremembered the 2nd entry, and thought we'd need to capture a comment on the 2nd issue that Symantec was excluded under), and also thought there'd be a keyword "closes" or "fixes". Thanks for working through these issues @adamfarley!

@sophia-guo sophia-guo merged commit 5ed95a8 into adoptium:master Jul 25, 2025
3 checks passed
@adamfarley
Copy link
Contributor Author

Thanks all 🤓

@adamfarley adamfarley deleted the remove_duplicate_excluded_tests branch July 25, 2025 13:24
@sxa
Copy link
Member

sxa commented Jul 28, 2025

Thanks. Ref slack thread about "ghost" failure on Linux/arm32 which this will have fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants