Fix detection of bundled JDK

imagej / imagej-launcher

The ImageJ native launcher

https://imagej.net/learn/launcher

BSD 2-Clause "Simplified" License

21 stars 23 forks source link

Fix detection of bundled JDK #60

Closed stelfrich closed 5 years ago

stelfrich commented 5 years ago

Since changing the logic for determining if a folder is a proper JDK/JRE in order to support Java9+, a system Java9+ would always take precedence over the bundled Java8 one. This PR fixes this behavior and adds an extremely inelegant hack to also make this work on macOS.

stelfrich commented 5 years ago

Merged this s.t. AppVeyor builds a new version. The situation can't really get worse than before.

ctrueden commented 5 years ago

Thanks @stelfrich!