Hi Erik,

On 11/06/2019 5:37 am, Erik Joelsson wrote:
Since JDK-8193255, when we started generating the cacerts file in the build, the build compare baseline builds have started failing. It seems the cacerts binary file has some non determinism built in so it doesn't get generated exactly the same given the same input. This patch adds special handling when comparing that file by comparing the output of "keytool -list" on the files instead.

Seems a reasonable approach.

Bug: https://bugs.openjdk.java.net/browse/JDK-8225392

Webrev: http://cr.openjdk.java.net/~erikj/8225392/webrev.01/

Code changes seem fine. I'm assuming this formulation doesn't run into the:

Warning: use -cacerts option to access cacerts keystore

that you get if you actually point keytool to the cacerts files in the JDK image:

> ./build/linux-x64-debug/images/jdk/bin/keytool -list -keystore build/linux-x64-debug/images/jdk/lib/security/cacerts -storepass changeit > certs.1
Warning: use -cacerts option to access cacerts keystore

Thanks,
David
-----

/Erik

Reply via email to