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

Add MSVS 2017 support for JDK11 and next to Windows platform #2792

Merged
merged 1 commit into from
Sep 14, 2018

Conversation

vsebe
Copy link
Contributor

@vsebe vsebe commented Sep 6, 2018

  • enable VS2017 as toolchain for JDK11
  • add MinGW to PATH

[ci skip]

The build environments table in the doc needs to be updated
https://www.eclipse.org/openj9/docs/openj9_support/#openjdk-10_1
eclipse-openj9/openj9-docs#72 (comment)

Signed-off-by: Violeta Sebe vsebe@ca.ibm.com

- enable VS2017 as toolchain for JDK11
- add MinGW to PATH

[ci skip]

Signed-off-by: Violeta Sebe <vsebe@ca.ibm.com>
@vsebe
Copy link
Contributor Author

vsebe commented Sep 6, 2018

Blocked by eclipse-omr/omr#2885

@vsebe
Copy link
Contributor Author

vsebe commented Sep 14, 2018

eclipse-omr/omr#2885 has been closed.
@pshipton are we ready to switch to VS2017?

@pshipton
Copy link
Member

pshipton commented Sep 14, 2018

are we ready to switch to VS2017?

I believe so, its supposed to work now.

@vsebe
Copy link
Contributor Author

vsebe commented Sep 14, 2018

Windows build passed.

@vsebe vsebe changed the title WIP: Add MSVS 2017 support for JDK11 and next to Windows platform Add MSVS 2017 support for JDK11 and next to Windows platform Sep 14, 2018
@vsebe
Copy link
Contributor Author

vsebe commented Sep 14, 2018

Jenkins test sanity win jdk11

@vsebe
Copy link
Contributor Author

vsebe commented Sep 14, 2018

@pshipton please review, thank you!

@pshipton
Copy link
Member

jenkins test extended win jdk11

@pshipton
Copy link
Member

The failures in the builds are expected and don't appear to be caused by the compiler change.

@pshipton pshipton merged commit 61324bc into eclipse-openj9:master Sep 14, 2018
@vsebe vsebe deleted the win_vs2017 branch October 3, 2018 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants