On Sat, 22 Feb 2025 14:40:39 GMT, Alexey Semenyuk <asemen...@openjdk.org> wrote:

>> Support the use of a custom msi wrapper executable when building an exe 
>> installer.
>> 
>> Put `installer.exe` file in the resource directory and jpackage will use it 
>> instead of the default `msiwrapper.exe` resource for exe installer.
>> 
>> To test this feature created a test that builds exe installer with a custom 
>> icon. The result installer exe is used as a custom msi wrapper executable in 
>> the second jpackage command that builds exe installer with the default icon. 
>> The installer exe produced by the second jackage command should have the 
>> same icon as the exe installer created in the first jpackage run.
>> 
>> Moved code verifying icons in executables from 
>> `LauncherIconVerifier.WinIconVerifier` class into 
>> `WinExecutableIconVerifier` class to make it available for tests. Replaced 
>> inline powershell script extracting icons from executables with standalone 
>> `read-executable-icon.ps1` powershell script. The script uses `ExtractIcon` 
>> instead of `ExtractAssociatedIcon`. It extracts icon from the executable's 
>> resources and will not fall back to anything if there is no icon resource.
>
> Alexey Semenyuk has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Keep bundles produces by the test in the test work directory

Thanks for explanation. It makes sense now and fix is just to provide 
workaround for such cases.

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

PR Comment: https://git.openjdk.org/jdk/pull/23732#issuecomment-2679293267

Reply via email to