RFR: 8316433: net.dll should delay load winhttp.dll [v2]
    Magnus Ihse Bursie 
    ihse at openjdk.org
       
    Thu Sep 21 10:09:42 UTC 2023
    
    
  
On Wed, 20 Sep 2023 03:51:20 GMT, Julian Waters <jwaters at openjdk.org> wrote:
>> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Revert test changes
>
> make/modules/java.base/Lib.gmk line 55:
> 
>> 53:     LDFLAGS := $(LDFLAGS_JDKLIB) \
>> 54:         $(call SET_SHARED_LIBRARY_ORIGIN), \
>> 55:     LDFLAGS_windows := -delayload:secur32.dll -delayload:iphlpapi.dll \
> 
> Wonder if this could be `LDFLAGS_microsoft_windows` to emphasize the compiler that this applies to
@TheShermanTanker I understand where you are coming from, but as a matter of fact we still only support building on Windows with Visual Studio, and as long as we do, we only need to specify "windows" in situations like these.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15793#discussion_r1332812580
    
    
More information about the build-dev
mailing list