-
-
Notifications
You must be signed in to change notification settings - Fork 249
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 support for Windows Redist DLL as Adoptium DevKit #3981
Conversation
Signed-off-by: Andrew Leonard <[email protected]>
Signed-off-by: Andrew Leonard <[email protected]>
Signed-off-by: Andrew Leonard <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd really prefer (even if we have this as a fallback) for it to be done in the playbooks so it is consistently on the machines instead of being dynamically downloaded. I don't really want to introduce more non-hermetic parts of the build process if possible.
@sxa
and will only pull if not on the build machine (except maybe I need a more Windows based path!) |
@sxa Where would you suggest for the local files on Windows ? |
Yeah To be honest unless we want to create some new directory under FYI @steelhead31 since he was involved in some of these discussions. |
Yes, I think given it's Windows, defaulting to say c:\openjdk as a local search location is probably fine. i'll update PR with that change, thanks |
Signed-off-by: Andrew Leonard <[email protected]>
Infra issue to extract onto Adoptium build nodes: adoptium/infrastructure#3772 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved now that the infrastructure issue is in flight and this will be a backup option for that :-)
Fixes #3887
Adds partial support for an Adoptium Windows DevKit, that consists of purely the Redist DLLs from https://github.com/adoptium/devkit-binaries/releases/tag/vs2022_redist_14.40.33807_10.0.26100.0
eg.
Test builds:
AQA Triage tests: adoptium/ci-jenkins-pipelines#1104 (comment)