RFR: JDK-8249289: Exception thrown when --temp points to non-existant directory

[alexander.matveev at oracle.com]

Hi Andy,

Can you double check two other places where File.list() was introduced 
with JDK-8223955 for similar issues?
One in MacPkgBundler.java and second in DeployParams.java.

Thanks,
Alexander

On 7/15/20 3:09 PM, Alexey Semenyuk wrote:
> Andy,
>
> Stream.close() call is missing on the result of Files.list() call.
> I'd also replace Files.list(root) with Files.walk(root, 1) call to 
> eliminate diving in directory tree.
>
> - Alexey
>
> On 7/15/2020 2:37 PM, Andy Herrick wrote:
>> Please review the jpackage fix for issue [1] at [2].
>>
>> This fixes the regression in JDK16 which caused exception when an 
>> empty directory was used int the --temp jpackage option, and added to 
>> the existing test.
>>
>> [1] - https://bugs.openjdk.java.net/browse/JDK-8249289
>>
>> [2[ - http://cr.openjdk.java.net/~herrick/8249289/webrev.02/
>>
>> /Andy
>>
>