please review revised webrev at:
http://cr.openjdk.java.net/~herrick/8251988/webrev.03/
jpackage.app-version set in cfg file, and jpackage.app-path now set from
native code.
New testcase in BasicTest to check that they are set.
/Andy
On 8/20/2020 7:36 PM, Alexey Semenyuk wrote:
Andy,
I'd replace 'File.separator' with '/' to make value of
'jpackage.app-path' property platform independent.
It would be also good to have a unit test verifying values of the new
properties.
- Alexey
On 8/20/2020 5:44 PM, Andy Herrick wrote:
Please review the jpackage fix at [1] to issue [2].
This fixes the problem loading libjli.dylib as well as providing some
useful generic system properties.
/Andy
[1] - http://cr.openjdk.java.net/~herrick/8251988/webrev.02/
<http://cr.openjdk.java.net/~herrick/8251988/webrev.01/>
[2] - https://bugs.openjdk.java.net/browse/JDK-8251988
