On Mon, 2 Dec 2024 15:10:08 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request with a new target base due 
>> to a merge or a rebase. The pull request now contains 25 commits:
>> 
>>  - Merge branch 'master' into static-jdk-image
>>  - Remove LDFLAGS_STATIC_JDK
>>  - Rename EXCLUDE_FROM_STATIC_LIBS to ONLY_EXPORTED, $(MODULE)_JDK_LIBS to 
>> $(MODULE)_INCLUDED_LIBS and module-libs.txt to module-included-libs.txt
>>  - Find LIBZIP_OBJS automatically
>>  - Find LIBJAVA_JPEG_OBJS automatically
>>  - Just exclude the a11y libraries from static builds
>>  - Copy debuginfo
>>  - Restore SetExecname in java_md.c
>>  - Fix incremental builds
>>  - Merge branch 'master' into static-jdk-image
>>  - ... and 15 more: https://git.openjdk.org/jdk/compare/c329f97f...8b1217a8
>
> src/hotspot/os/linux/os_linux.cpp line 590:
> 
>> 588: 
>> 589:     // Found the full path to the binary. It is normally of this 
>> structure:
>> 590:     //   <jdk_path>/lib/<hotspot_variant>/libjvm.so
> 
> You could change <hotspot_variant> to server I believe, since there isn't a 
> client (I hope) version anymore.

Variant can also be e.g. `minimal`. And I do think the client variant is still 
supported, even if it is not actively built..?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20837#discussion_r1866028280

Reply via email to