Looks good to me as well, but sadly I have not had a chance to test it.

Tim

On 05/28/13 04:19 AM, Staffan Larsen wrote:
Looks good. I have verified that this patch signs the binaries as expected.

Thanks,
/Staffan

On 28 maj 2013, at 13:15, Erik Joelsson<[email protected]>  wrote:

This patch readds the missing functionality of running codesign on certain 
launchers on macosx. In the old build, it was always run, but failing if the 
openjdk_codesign certificate wasn't present on the system. Make ignored this 
failure. In this patch, configure tests that the certificate is there and 
disables running codesign if it isn't, reducing pointless error messages in the 
build log.

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

/Erik

Reply via email to