[PATCH] 8004789 : (zipfs) zip provider doesn't work correctly with file systems providers rather than the default

[Alan Bateman]

On 09/12/2012 22:08, Philippe Marschall wrote:
> Hi
>
> As discussed previously [1] here are the patches against JDK 7 and 8
> to make the ZipFileSystem provider use Files#createTempFile instead of
> File#createTempFile. This causes the temporary file to be created on
> the same file system as the ZIP file resides on. This makes it
> possible to run the zipfs provider on a file system other than the
> default one.
>
> This patch does not include any tests but I also could not find any
> existing ones. If somebody wants tests I'd be happy to supply them
> once instructed who to do them and where to put them.
>
Thanks for the patch, it looks okay to me. I see that toAbsolutePath is 
used but that isn't actually required. Sherman - do you want to sponsor 
this?

The tests for the zip provider are in jdk/test/demo/zipfs. As the 
scenario requires three file system providers to demonstrate then it 
would be a lot of work to create a test case, in which case I think it's 
okay to let it out for this one.

-Alan