Looks fine to me, too. Are you going to add any tests for this case?
-- Kevin
On 2/19/2019 12:11 PM, Alexander Matveev wrote:
Hi Andy,
Looks good.
Thanks,
Alexander
On 2/19/2019 7:59 AM, Andy Herrick wrote:
JDK-8217798: modular jar linking in jpackage
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).
[1] https://bugs.openjdk.java.net/browse/JDK-8217798
[2] http://cr.openjdk.java.net/~herrick/8217798/
/Andy Herrick
