RFR: 8346719: Add relaunchers to the static JDK image for missing executables [v3]
Magnus Ihse Bursie
ihse at openjdk.org
Tue Sep 9 13:15:58 UTC 2025
On Mon, 11 Aug 2025 17:20:30 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
>> This solution was modeled on how we create `module-included-libs.txt`. I agree that it is a bit hacky to inject this kind of stuff in ModuleWrapper, but I can't see how we can do it more cleanly. That's the only point at which we know the entire module for a phase.
>>
>> Maybe we can improve optics by doing a more official-looking "hook" for injecting functionality at pre-/post- whole-module processing? That would keep ModuleWrapper slimmer and without all the current specialized hooks, and make the name "wrapper" more relevant.
>>
>> Or, we could rename the makefile to indicate better that it does a lot more than just wraps a file. (I have no good suggestions right now.)
>
> I'm not sure what you mean here. I described a solution that keeps it in LauncherCommon without creating races.
@erikj79 I've finally gotten around to fixing this. Looks ok?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24380#discussion_r2333542410
More information about the core-libs-dev
mailing list