I guess it was a type in webrev adderss. Should be
http://cr.openjdk.java.net/~herrick/8230920/webrev.03/
This webrev looks good.
- Alexey
On 9/29/2019 10:08 AM, Andy Herrick wrote:
Please review the revised jpackage fix for bug [1] at [3].
This is a fix for the JDK-8200758-branch branch of the open sandbox
repository (jpackage).
The revised fix stores an xml file (.jpackage.xml) in the app-image so
that jpackage commands using that app-image can determine some things
previously available only when building the app-image, such as the
name of the application and the name of additional launchers.
Initially this is only used on windows, but should be later used on
linux and possibly macOS as well (if additional data needed by dmg or
pkg building are identified).
The windows fix also ensures that shortcuts are created for all
launchers (when shortcut hint option(s) are used).
[1] https://bugs.openjdk.java.net/browse/JDK-8230920
[3] http://cr.openjdk.java.net/~herrick/8230920/webrev.01/
/Andy
On 9/24/2019 8:54 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).
This fix replaces the practice we were using on windows to determine
the application name, and name of additional launchers (by looking
for ".cfg" files in the app dir.
Instead we now add a file ".jpackage.args" to the root of the
app-image, and record in that file all the arguments used to create
the app-image. We later read that file to determine the original app
name and any additional launcher names.
This change also fixes the shortcut creation on windows to create
shortcuts (if so directed) for all launchers.
[1] https://bugs.openjdk.java.net/browse/JDK-8230920
[2] http://cr.openjdk.java.net/~herrick/8230920/webrev.01/
/Andy
