On Wed, 10 Apr 2024 21:10:27 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

>> This is the pinnacle of the recent stream of refactorings in the build 
>> system. This patch introduces a more abstract concept of "JDK_LIBS", where 
>> only the library name (e.g. "java" or "java.desktop:jawt") is specified, and 
>> the build system turns this into suitable linker flags: `-ljawt -L<correct 
>> path>` or `jawt.lib -libpath:<correct path>`, depending on linker. It will 
>> also automatically create proper dependencies.
>
> Magnus Ihse Bursie has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fix missing lib prefix

make/common/JdkNativeCompilation.gmk line 212:

> 210: 
> 211:   ifneq ($(STATIC_LIBS), true)
> 212:     ifeq ($$(call isTargetOs, windows), true)

I should've looked through this more carefully. The selection on whether to use 
-L and -libpath: should've been based on the compiler, not the target OS

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18649#discussion_r1565162835

Reply via email to