Re: RFR: JDK-8217331: Problems when space in application name

2019-01-28 Thread Alexander Matveev
Hi Andy, Looks fine. Thanks, Alexander On 1/27/2019 4:56 AM, Andy Herrick wrote: Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). The webrev includes removal of the APP_FS_NAME DeployParam, and

Re: RFR: JDK-8217331: Problems when space in application name

2019-01-28 Thread Kevin Rushforth
This looks fine to me. I note that it still allows characters that will cause problems (e.g., ":" on Linux or ";" on Windows), but that could be done as a follow-on bug. -- Kevin On 1/27/2019 4:56 AM, Andy Herrick wrote: Please review the jpackage fix for bug [1] at [2]. This is a fix for

RFR: JDK-8217331: Problems when space in application name

2019-01-27 Thread Andy Herrick
Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). The webrev includes removal of the APP_FS_NAME DeployParam, and enhanced  checking of the given application name so APP_NAME parameter can be safely