RFR: 8171508: Remove -Dsun.java.launcher.is_altjvm option [v3]
David Holmes
dholmes at openjdk.org
Sun Apr 6 22:11:49 UTC 2025
On Thu, 3 Apr 2025 22:39:07 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> This change fixes two issues in `os::jvm_path()`:
>>
>> - go back 4 instead of 5 slashes and then checking for "/jre/lib";
>> - fix the hardcoded "hotspot" sub-directory before the jvm library name.
>>
>> Passed tiers 1 - 3 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
>
> @dholmes-ora comment
Also as discussed offline picking up JAVA_HOME from the environment is not the right thing to do in general. It only worked with -Dsun.java.launcher.is_altjvm because we ensured we set JAVA_HOME when using the property.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24310#issuecomment-2781695926
More information about the hotspot-runtime-dev
mailing list