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

Guard jvmtitests newly added header includes to J9ZOS only #20289

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

Deigue
Copy link
Contributor

@Deigue Deigue commented Oct 2, 2024

Have draft PR ready here @r30shah
Double checking right now the below, before I remove it from draft:

  • full vm open-xl build
  • java 8 build (would like more insight on what all platforms we should verify here to minimize surprises later)

@keithc-ca keithc-ca changed the title Guards jvmtitests newly added header includes to J9ZOS only Guard jvmtitests newly added header includes to J9ZOS only Oct 2, 2024
Signed-off-by: Gaurav Chaudhari <gaurav.chaudhari@ibm.com>
@Deigue
Copy link
Contributor Author

Deigue commented Oct 2, 2024

Updated commit message to match title.

@keithc-ca
Copy link
Contributor

Can you mark this as ready for review? Windows builds will be broken until this is merged.

@Deigue Deigue marked this pull request as ready for review October 3, 2024 16:27
@keithc-ca keithc-ca merged commit 2e44afd into eclipse-openj9:master Oct 3, 2024
2 checks passed
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.

2 participants