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

Include JDK17 in OMR coordinated merge documentation #16004

Merged
merged 1 commit into from
Sep 28, 2022

Conversation

0xdaryl
Copy link
Contributor

@0xdaryl 0xdaryl commented Sep 28, 2022

Signed-off-by: Daryl Maier maier@ca.ibm.com

Signed-off-by: Daryl Maier <maier@ca.ibm.com>
@0xdaryl
Copy link
Contributor Author

0xdaryl commented Sep 28, 2022

@pshipton : would you mind merging?

I only added JDK17 to the list of build targets for the OMR acceptance build. The rest of the changes are automatic whitespace removal at end-of-lines.

@pshipton
Copy link
Member

pshipton commented Sep 28, 2022

I expect it will become important to check a more recent version as well, i.e. jdk19 or later, once it's ready. But that can be added later. The OMR acceptance build doesn't consider it now.

@pshipton pshipton merged commit 2594799 into eclipse-openj9:master Sep 28, 2022
@pshipton
Copy link
Member

pshipton commented Sep 28, 2022

The OMR acceptance build doesn't consider it now.

We are getting closer for jdk19, I'll modify the acceptance build now and we can see how it goes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants