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

Update JDK8/J9/Win to use VS2013 instead of VS2010 #2191

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

sxa
Copy link
Member

@sxa sxa commented Oct 29, 2020

Requested via #466 (comment)

Signed-off-by: Stewart X Addison sxa@redhat.com

Signed-off-by: Stewart X Addison <sxa@redhat.com>
@sxa sxa added enhancement Issues that enhance the code or documentation of the repo in any way windows Issues that affect or relate to the WINDOWS OS openj9 Issues that are enhancements or bugs raised against the OpenJ9 group labels Oct 29, 2020
@sxa sxa added this to the October 2020 milestone Oct 29, 2020
@sxa sxa self-assigned this Oct 29, 2020
@sxa
Copy link
Member Author

sxa commented Oct 29, 2020

Validating at https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/918/OS=Win2012,label=vagrant/

(Result: Tried to activate 2013 and fails as a result of issues with the rebase command which is changed in the upstream OpenJ9 PRs which this has to be merged simultaneously with)

@sxa
Copy link
Member Author

sxa commented Oct 29, 2020

NOTE: Merging of this needs to be co-ordinated with OpenJ9 since it has to be synced with the merging of ibmruntimes/openj9-openjdk-jdk8#449 and eclipse-openj9/openj9#10892

Copy link
Contributor

@karianna karianna left a comment

Choose a reason for hiding this comment

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

LGTM

@sxa sxa marked this pull request as ready for review October 29, 2020 18:23
@adoptopenjdk-github-bot
Copy link
Contributor

🟢 PR TESTER RESULT 🟢

✅ All pipelines passed! ✅

@sxa
Copy link
Member Author

sxa commented Nov 4, 2020

Note to reviewers: Failure on the checks is a macos compiler related error, so unrelated to this PR

@sxa
Copy link
Member Author

sxa commented Nov 4, 2020

Both upstream changes now merged, so merging this one.

@sxa sxa merged commit 1d17bac into adoptium:master Nov 4, 2020
sxa added a commit to sxa/temurin-build that referenced this pull request Nov 18, 2020
karianna pushed a commit that referenced this pull request Nov 25, 2020
@sxa sxa deleted the jdk8j9_vs2013 branch December 24, 2020 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issues that enhance the code or documentation of the repo in any way openj9 Issues that are enhancements or bugs raised against the OpenJ9 group windows Issues that affect or relate to the WINDOWS OS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants