On Wed, 4 Dec 2024 07:31:24 GMT, Henry Jen <[email protected]> wrote:
>> Add jlink plugins to allow branding change for java.vendor, java.vm.vendor
>> and java.vendor.url.
>>
>> The jlink plugin will change the value in java.lang.VersionProps, which will
>> set those property values. The `java.vm.vendor` was initialized by VM with
>> value set at build time, and then later be replaced with value from
>> VersionProps.
>>
>> To keep current behavior, we treat 'N/A' value as no-op to mimic current
>> build behavior. Perhaps we don't really need this, as proper value should be
>> set with `branding.conf` in official build.
>
> Henry Jen has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Clean up adapting review comments
src/java.base/share/classes/java/lang/VersionProps.java.template line 128:
> 126:
> 127: // In case VM is not yet set this property
> 128: props.putIfAbsent("java.vm.vendor", VENDOR_VM);
Can this system property just be set here for simplicity? The value of
`java.vendor` and `java.vm.vendor` system property are the same.
The VM does not need this value until VM initialization completes. It's only
needed by StatSampler (jstat).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21964#discussion_r1870070455
