RFR(XXS) : 8158185 : jdk/test/lib/FileInstaller throws NPE if dst is in current directory

[Vladimir Kozlov]

Good. Thank you.

Vladimir

On 5/31/16 6:29 AM, Igor Ignatyev wrote:
> Hi Vladimir,
>
> 1st of all, thank you for reviewing all these changes!
>
> regarding testing, I run all changed tests locally on mac and run all hotspot/compiler tests remotely on all supported platforms in fastdebug, product variant w/ Xcomp,Xmixed.
> this particular fix was tested as a part of JDK-8158184 testing.
>
> Thanks,
> — Igor
>> On May 31, 2016, at 5:14 AM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>
>> Looks good.
>>
>> Igor, how are you testing all these tests fixes?
>>
>> Thanks,
>> Vladimir
>>
>> On 5/30/16 5:08 PM, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev/8158185/webrev.00/
>>>> 2 lines changed: 0 ins; 0 del; 2 mod;
>>>
>>> Hi all,
>>>
>>> could you please review this small changeset which fixes FileInstaller? FileInstaller throws NPE if 2nd argument (dst) is in current directory, this happens because Path::getParent returns null if Path doesn’t have a parent. to prevent this, I added Path::toAbsolutePath call.
>>>
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8158185
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8158185/webrev.00/
>>>
>>> Thanks,
>>> — Igor
>>>
>