RFR: JDK-8230519: jpackage "--package-type" values and default
Kevin Rushforth
kevin.rushforth at oracle.com
Mon Sep 9 17:14:23 UTC 2019
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
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