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