Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Cheesebaron
Copy link

@Cheesebaron Cheesebaron commented Oct 23, 2025

GitHub Issue: closes #
Fixes #463

PR Type:

  • 🐞 Bugfix

What is the current behavior? 🤔

  • Uno.Check has hardcoded path on macOS to look for microsoft-11 JDK distribution
  • It checks folder Library/Java/JavaVirtualMachines but should be /Library/Java/JavaVirtualMachines
  • Only checks for microsoft and openjdk distributions
    • This wouldn't allow me to use temurin which I have installed

What is the new behavior? 🚀

  • Removed hardcoded microsoft-11 path
  • Check correct /Library/Java/JavaVirtualMachines folder for installed JDKs
  • Added temurin, zulu and corretto as allowed JDKs too (perhaps all open JDK distributions should be allowed?)

PR Checklist ✅

Please check if your PR fulfills the following requirements:

Other information ℹ️

@CLAassistant
Copy link

CLAassistant commented Oct 23, 2025

CLA assistant check
All committers have signed the CLA.

@Cheesebaron Cheesebaron force-pushed the dependencies/update-macos-java-dir branch from d1b7ee4 to 86510d9 Compare October 23, 2025 08:02
@Cheesebaron Cheesebaron changed the title Fix JDK checks on macOS fix: JDK checks on macOS Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[.NET 10][Mac] Build fails for Android

2 participants