RFR: JDK-8246010: AdditionalLaunchersTest is not enabled, and fails.
Alexey Semenyuk
alexey.semenyuk at oracle.com
Fri May 29 23:16:31 UTC 2020
Andy,
The check is added to WindowsAppImageBuilder.java which means it will be
done for app image bundling too. However the problem is with msi
bundling only as far as I understood from your explanation.
Did you try fix suggested at [1]?
[1]
https://stackoverflow.com/questions/15431210/wix-relative-path-ends-up-being-too-long
- Alexey
On 5/29/2020 7:12 PM, Andy Herrick wrote:
>
> On 5/29/20 7:00 PM, alexander.matveev at oracle.com wrote:
>> Hi Andy,
>>
>> Why we need error.lock-resource?
> because resource was used , but never defined.
>>
>> Also, I do not like 194 limit obtained by testing, since once new
>> files are introduced in runtime this value might not work anymore.
>> Can we check on actual file paths to make sure they have valid length?
>
> I don't like it either, but otherwise we need to test length every
> time we create a path, and the check needs to be platform dependent.
>
> /Andy
>
>>
>> Thanks,
>> Alexander
>>
>> On 5/29/20 7:36 AM, Andy Herrick wrote:
>>> Please review jpackage fix at [1] to issue [2]
>>>
>>> /Andy
>>>
>>> [1] http://cr.openjdk.java.net/~herrick/8246010/webrev.01/
>>>
>>> [2] https://bugs.openjdk.java.net/browse/JDK-8246010
>>>
>>
More information about the core-libs-dev
mailing list