Re: [9] Review request: JDK-8147781: Javapackager installer needs to cleanup temporary folders

Thanks for pointing out the problem. Currently, the directory is not removed regardless "-verbose" is specified or not. I think we can check if "-verbose" is specified, and if it is not, remove the temporary directory. New webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8147781/webrev.01/

Re: [9] Review request: JDK-8147781: Javapackager installer needs to cleanup temporary folders

That’s a good point Stefan. I hadn’t noticed the -verbose. Chris > On Feb 6, 2017, at 4:31 PM, Stefan Fuchs wrote: > > Hi, > > are you sure that there is a bug? > > The verbose paramter is explicitly documented to leave temporary files > around, as a starting point for

Re: [9] Review request: JDK-8147781: Javapackager installer needs to cleanup temporary folders

Hi, are you sure that there is a bug? The verbose paramter is explicitly documented to leave temporary files around, as a starting point for customization. Quote from https://docs.oracle.com/javase/8/docs/technotes/guides/deploy/self-contained-packaging.html Verbose mode includes the

[9] Review request: JDK-8147781: Javapackager installer needs to cleanup temporary folders

Chris, Please review the changes about removing temporary files. JIRA: https://bugs.openjdk.java.net/browse/JDK-8147781 Webrev: http://cr.openjdk.java.net/~vdrozdov/JDK-8147781/webrev.00/ --Victor