RFR: JDK-8215019: Allow --install-dir on windows
Alexander Matveev
alexander.matveev at oracle.com
Mon Mar 25 22:35:44 UTC 2019
Hi Andy,
http://cr.openjdk.java.net/~almatvee/8215019/webrev.01/
Updated webrev with added missing message in
HelpResources_zh_CN.properties and updated messages below:
MSG_Help_win_install_dir=\
\Relative sub-path under the default installation location\n\
message.invalid.install.dir=Warning: Invalid install directory {0}.
Install directory should be a relative sub-path under the default
installation location such as "Program Files". Defaulting to application
name "{1}".
Thanks,
Alexander
On 3/21/2019 10:08 AM, Andy Herrick wrote:
> I think you missed HelpResources_zh_CN.properties
>
> I think wording of help text: 'sub-path of the installation location
> of the application such as"Program Files" or "AppData"' is
> misleading. It reads like "Program files" or "AppDAta" are examples
> of the sub-path.
>
> The error message says: 'sub-path under default installation location"
> which is a little better. (though there should be a "the" before
> default).
>
> /Andy
>
>
> On 3/20/2019 8:05 PM, Alexander Matveev 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).
>>
>> - Added support for --install-dir on Windows. It should be relative
>> path to "Program Files" or "AppData".
>> - If --install-dir is invalid we will use app name instead.
>> - Added two new tests to validate --install-dir and related
>> functionality.
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8215019
>>
>> [2] http://cr.openjdk.java.net/~almatvee/8215019/webrev.00/
>>
>> Thanks,
>> Alexander
>
More information about the core-libs-dev
mailing list