RFR: 8171508: Remove -Dsun.java.launcher.is_altjvm option [v6]
Thomas Stuefe
stuefe at openjdk.org
Wed Apr 9 06:38:33 UTC 2025
On Wed, 9 Apr 2025 06:19:52 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:
>> The `-Dsun.java.launcher.is_altjvm` option is only used in the gtest launcher and has no relationship with the similarly named `-XXaltjvm` option. The gtest launcher also sets the `-XX:+ExecutingUnitTests` option. This RFE involves removing the `-Dsun.java.launcher.is_altjvm` option, capturing the `-XX:+ExecutingUnitTests` option in Arguments::process_sun_java_launcher_properties() so that the os::jvm_path() can check if the option is set before using JAVA_HOME setting.
>>
>> Passed tiers 1 - 3 testing.
>
> Calvin Cheung has updated the pull request incrementally with one additional commit since the last revision:
>
> simplify the code by using stringStream
Nice, this looks good to me. Thank you.
-------------
Marked as reviewed by stuefe (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/24310#pullrequestreview-2752209598
More information about the hotspot-runtime-dev
mailing list