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

Alexey Semenyuk alexey.semenyuk at oracle.com
Mon Sep 9 17:32:35 UTC 2019



On 9/9/2019 1:28 PM, Andy Herrick wrote:
>
> 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 ?
I think exiting with error if dmg tools are not available on macOS would 
be better. This would be more predictable.

- Alexet
>
>
> /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,
>>>
>>



More information about the core-libs-dev mailing list