looks good.
/Andy
On 9/12/2019 5:44 PM, Alexander Matveev wrote:
Hi Alexey,
http://cr.openjdk.java.net/~almatvee/8230629/webrev.01
I simplified isFileSigned() as you suggested.
Thanks,
Alexander
On 9/12/2019 4:13 AM, Alexey Semenyuk wrote:
Looks good, though
---
List args = new
Thank you, looks good.
- Alexey
On 9/12/2019 5:44 PM, Alexander Matveev wrote:
Hi Alexey,
http://cr.openjdk.java.net/~almatvee/8230629/webrev.01
I simplified isFileSigned() as you suggested.
Thanks,
Alexander
On 9/12/2019 4:13 AM, Alexey Semenyuk wrote:
Looks good, though
---
List args =
Hi Alexey,
http://cr.openjdk.java.net/~almatvee/8230629/webrev.01
I simplified isFileSigned() as you suggested.
Thanks,
Alexander
On 9/12/2019 4:13 AM, Alexey Semenyuk wrote:
Looks good, though
---
List args = new ArrayList<>();
940 args.addAll(Arrays.asList("codesign",
941
Looks good, though
---
List args = new ArrayList<>();
940 args.addAll(Arrays.asList("codesign",
941 "--verify",
942 file.toString()));
943
944 ProcessBuilder pb
945 = new ProcessBuilder(args.toArray(new
String[args.size()]));
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).
- Binaries in runtime and Frameworks will not be signed directly using
user provided certificate.
- libapplauncher.dylib will be signed with user
