RFR: 8171508: os::jvm_path: error in handling -Dsun.java.launcher.is_altjvm option after 8066474 [v2]

[David Holmes]

On Thu, 3 Apr 2025 04:52:27 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - @tstuefe and @dholmes-ora comments
>  - Merge branch 'master' into 8171508-os-jvm_path
>  - 8171508: os::jvm_path: error in handling -Dsun.java.launcher.is_altjvm option after 8066474

This issue should be renamed "Remove -Dsun.java.launcher.is_altjvm option". That requires a CSR request to be filed.

Once again for the record -XXaltjvm has nothing to do with the -Dsun.java.launcher.is_altjvm option.

https://mail.openjdk.org/pipermail/hotspot-runtime-dev/2017-April/023241.html

Removal of  -XXaltjvm would also need a CSR request.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/24310#issuecomment-2774715610