Looks good.
- Alexey
On 6/6/2019 3:48 PM, Andy Herrick wrote:
OK - revised MacCertificate as per below:
webrev: http://cr.openjdk.java.net/~herrick/8223333/webrev.03
/Andy
On 6/6/2019 12:47 PM, Alexey Semenyuk wrote:
http://cr.openjdk.java.net/~herrick/8223333/webrev.02/src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacCertificate.java.sdiff.html,
lines 81-93 can be replaced with a single statement, something like:
---
java.nio.file.Files.copy(
new ByteArrayInputStream(baos.toByteArray()),
output.toPath(),
StandardCopyOption.REPLACE_EXISTING);
---
- Alexey
On 6/5/2019 7:54 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).
[1] https://bugs.openjdk.java.net/browse/JDK-8223333
[2] http://cr.openjdk.java.net/~herrick/8223333/
/Andy