RFR: 8171508: Remove -Dsun.java.launcher.is_altjvm option [v3]
David Holmes
dholmes at openjdk.org
Mon Apr 7 06:08:57 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
src/hotspot/os/aix/os_aix.cpp line 1326:
> 1324: }
> 1325:
> 1326: // modules image doesn't have "jre" subdirectory
Delete comment please
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24310#discussion_r2028155906
More information about the hotspot-runtime-dev
mailing list