On Tue, 19 Nov 2024 22:07:10 GMT, Alexey Semenyuk <asemen...@openjdk.org> wrote:

>> Fix `ResourceEditor` class to handle long paths.
>> 
>> Based on https://github.com/openjdk/jdk/pull/9384 PR.
>> 
>> This repairs app image building on Windows if the path to jpackage temp 
>> directory exceeds 260 (`MAX_PATH`) characters. MSI building is still broken 
>> and can NOT be repaired due to WiX's limitations.
>> 
>> Repaired scenario when the `--dest` option references a long path on 
>> Windows. Previously jpackage would have failed to build MSI package.
>> 
>> Other changes:
>>  - Rename `Executor.setRemovePath()` into `Executor.setRemovePathEnvVar()` 
>> to make the method name match the implementation.
>>  - Move the mutable part of `WixPipeline` class into `WixPipeline.Builder` 
>> class.
>>  - Fixed minor flaws in "WinL10nTest.java"
>
> Alexey Semenyuk has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Make path guaranteed to be long enough

Marked as reviewed by almatvee (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/22092#pullrequestreview-2447250167

Reply via email to