RFR: JDK-8230519: jpackage "--package-type" values and default

[Andy Herrick]

On 9/9/19 1:14 PM, Kevin Rushforth wrote:
> Looks good with one question:
>
> In Arguments.java:
>
> +                     if (bundler.isDefault()) {
> +                         return bundler;
> +                     } else {
> +                         savedBundler = bundler;
> +                     }
>
> When would there be a valid case where you loop through the list of 
> bundlers and don't find a default? It may be better to throw an error 
> in that case rather than just return the last one found.
>
> -- Kevin
>
This left the possibility that if the default were not supported but 
another form was, then it became the default.

On Windows, the two package types, msi and exe now require the same 
tools, and on linux, deb is only the default if (among other things) it 
is supported.

So that leaves only macOS.  here dmg is default but required hdmiutil 
and osascript.

If either of these are missing dmg is not supported, so this code would 
build pkg package if --package-type not specified.  Would it be better 
to just get error and only build pkg if it is explicitly specified as 
--package-type pkg ?


/Andy

>
> On 9/8/2019 2:50 PM, 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).
>>
>> This fix:
>>
>> Adds "app-image" as a valid value for "--package-type" options, 
>> meaning build an application image instead of a package.
>>
>> Changes the default value of "--package-type" to a platform dependent 
>> default package type.
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8230519
>> [2] http://cr.openjdk.java.net/~herrick/8230519/webrev.01/
>>
>> Thanks
>>
>> Andy,
>>
>