On Wed, 22 Oct 2025 04:36:25 GMT, Alexey Semenyuk <[email protected]> wrote:

> Don't put the default icon (.png) files in the app image on Linux.
> 
> A simple fix would be to add a condition to 
> `LinuxPackagingPipeline.writeLauncherIcons()` without changing interfaces in 
> the model. However, this would leave unaddressed a long-standing issue with 
> `ApplicationImageUtils.createLauncherIconResource()` function that requires 
> the main launcher to calculate an icon resource for an additional launcher. 
> At the packaging phase, any `Launcher` instance should be self-sufficient. 
> Fixing `ApplicationImageUtils.createLauncherIconResource()` required adding a 
> new interface `ResourceDirLauncherIcon` to the model and changing the 
> semantics of the `DefaultLauncherIcon` interface.
> 
> A new `ApplicationBuilder.normalizeLauncherProperty()` function has been 
> added to calculate the value of an additional launcher's property from the 
> value of this property in the main launcher. The function is supposed to 
> calculate a description, shortcuts, an icon, and any future launcher 
> property. For now, it is used to calculate the launcher icon.

This pull request has now been integrated.

Changeset: 327b7c3c
Author:    Alexey Semenyuk <[email protected]>
URL:       
https://git.openjdk.org/jdk/commit/327b7c3cd854bea3ffce557b981df535d5ed04bf
Stats:     565 lines in 25 files changed: 431 ins; 49 del; 85 mod

8370100: Redundant .png files in Linux app-image cause unnecessary bloat

Reviewed-by: almatvee

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

PR: https://git.openjdk.org/jdk/pull/27928

Reply via email to