On Thu, 19 Dec 2024 16:27:30 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

> We need to raise the abstraction of the SetupBuildLauncher API, to prepare 
> for static launchers. We should specify the desired outcome, not what flags 
> we should add. This can be seen as the last part of 
> [JDK-8141444](https://bugs.openjdk.org/browse/JDK-8141444) (9 years later).
> 
> In the process, I am removing EXTRA_JAVA_ARGS which has not been used for a 
> long time. I also removed this part from the launcher itself.

I'm not convinced this level of abstraction makes things better. It adds quite 
a bit of overhead for adding additional customization options for when adding 
new launchers. Could you elaborate more on why this is needed?

make/common/modules/LauncherCommon.gmk line 162:

> 160:           -framework ApplicationServices \
> 161:           -framework Cocoa \
> 162:           -framework Security, \

What happened to all of these? Were they never needed?

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

PR Review: https://git.openjdk.org/jdk/pull/22832#pullrequestreview-2516304227
PR Review Comment: https://git.openjdk.org/jdk/pull/22832#discussion_r1893193068

Reply via email to