RFR: 8339480: Build static-jdk image with a statically linked launcher [v13]

Magnus Ihse Bursie ihse at openjdk.org
Mon Nov 18 11:10:09 UTC 2024


On Tue, 3 Sep 2024 19:51:39 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Also include information about where generated data is consumed.
>>  - Document how and why we keep track of native libraries per module
>
> make/ModuleWrapper.gmk line 59:
> 
>> 57:   endif
>> 58: endif
>> 59: 
> 
> This part looks a bit convoluted. It would be nice with a comment explaining what it does, where `$($(MODULE)_JDK_LIBS)` is coming from and what consumes the module-libs.txt.

Fixed.

> make/common/JdkNativeCompilation.gmk line 310:
> 
>> 308:       $$(MODULE)_JDK_LIBS += $$($1_NAME)
>> 309:     endif
>> 310:   endif
> 
> Same, here as in ModuleWrapper.gmk, I think we need a comment explaining how this is consumed.

Fixed.

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

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


More information about the build-dev mailing list