Review request : 7031166 : pack200 CommandLineTest test failure - simple review

[Alan Bateman]

Kumar Srinivasan wrote:
> Hi,
>
> Replaces some old nio code which copies over the test sdk to run tests 
> against.
> http://cr.openjdk.java.net/~ksrini/7031166/webrev.0/
>
> Thanks
> Kumar
I see the original code creates the parent directories if they don't 
exist so if you want it work the same way then you could change it to:

Path parent = dst.toPath().getParent();
if (parent != null)
    Files.createDirectories(parent);
Files.copy(....);

I see that CommandLineTests.runPack200 gets the absolute path before 
calling this function. You shouldn't need to do that now. I also see it 
tests if the target file exists and deletes it before the copy. You can 
skip that too if you add the REPLACE_EXISTING option to the copy. One 
other suggestion is to do a static import of StandardCopyOption to avoid 
the copy going into a second line.

-Alan.