RFR: 8339480: Build static-jdk image with a statically linked launcher [v7]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Nov 18 10:42:33 UTC 2024
On Mon, 21 Oct 2024 13:07:34 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> src/java.base/unix/native/libjli/java_md.c line 279:
>>
>>> 277: char jvmpath[], jint so_jvmpath,
>>> 278: char jvmcfg[], jint so_jvmcfg) {
>>> 279: /* Compute/set the name of the executable. This is needed for macOS. */
>>
>> But this file is not used when on macOS, is it? (cfr java_md_macosx.m) )
>
> Hm. I think both are used? I'll need to double-check that.
You are right. Don't know how this got in here, I must have thought it needed at some point when trying to get this to work on macOS.
This really is a sign that I've been doing cargo-cult programming. The entire JDK bootstrapping mechanism is so spread-out and difficult to follow that it ends up being more "poke a bit here and see if it works" kind of programming instead of understanding what needs to be done and doing it. We should really make a follow-up refactoring on this to get it to a better state, across all modules and components...
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20837#discussion_r1846348477
More information about the core-libs-dev
mailing list