Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude hotspot runtime tests not supported on windows aarch64 #5910

Merged

Conversation

Haroon-Khel
Copy link
Contributor

See #5909

2 hotspot tests are not supported on windows aarch64

Will merge into release branch after merged into master

@Haroon-Khel
Copy link
Contributor Author

Someone with write access will need to merge this

@smlambert
Copy link
Contributor

Please see #5887 and cherry pick it

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the issue this is being excluded under (see #5886), and sorry, I did not exclude already for JDK23

@andrew-m-leonard
Copy link
Contributor

Please update the issue this is being excluded under (see #5886), and sorry, I did not exclude already for JDK23

@Haroon-Khel This also fails on jdk-21, please can you add to jdk21 exclude as well?

@andrew-m-leonard
Copy link
Contributor

Please update the issue this is being excluded under (see #5886), and sorry, I did not exclude already for JDK23

@Haroon-Khel This also fails on jdk-21, please can you add to jdk21 exclude as well?

@Haroon-Khel scratch that, it's already excluded on jdk-21 master branch

@karianna
Copy link
Contributor

@Haroon-Khel I can merge this but wanted to make sure you caught the Q's from @smlambert

@sophia-guo sophia-guo merged commit 1c9ec21 into adoptium:master Jan 27, 2025
3 checks passed
@sophia-guo
Copy link
Contributor

Might be better to also exclude for jdk23+

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.

6 participants