Skip to content

Added in support for Microsoft's Build of OpenJDK #3723

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

thozook
Copy link

@thozook thozook commented Jan 31, 2022

Resolves issue reported in #3719

Used the previous "$jvmHotSpot" variable since the registry key for Microsoft's OpenJDK build is also in the same hotspot subdirectory.

@anatolybolshakov
Copy link
Contributor

@thozook thanks! Let us take a look

@EzzhevNikita EzzhevNikita requested a review from a team February 2, 2022 15:41
@AndreyIvanov42
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@hymccord
Copy link

hymccord commented Dec 15, 2022

@anatolybolshakov Any movement on this PR?
I build self-hosted agent images using https://github.com/actions/runner-images
These images use the Microsoft OpenJDK and the agent does not add JAVA as a system capability. I have to add it manually.

I would edit the powershell files myself, but those changes would be wiped out on agent upgrade correct?

@anatolybolshakov
Copy link
Contributor

Hi @hymccord I'm no longer working on project - @kirill-ivlev would probably be the right person to address

@LarsLaskowski
Copy link

#3719 is closed, but the issue is still available. I think it makes sense that the Microsoft OpenJDK installation is also recognized by the agent. I just had to go back to Adoptium Eclipse today because it doesn't work. What a pity.

@MichelZ
Copy link

MichelZ commented Jul 23, 2024

@aleksandrlevochkin @kirill-ivlev I was having an issue with a self-hosted agent and Microsoft Java recently. This would have prevented the issue I think. Can this be merged?

@bdovaz
Copy link

bdovaz commented Aug 30, 2024

Any updates? Thanks.

Related #3624

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.

9 participants