Re: RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-24 Thread Jia Huang
Thanks Erik. 在 2019年09月24日 23:19, Erik Joelsson 写道: Sure, will do. /Erik

Re: RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-24 Thread Erik Joelsson
Sure, will do. /Erik On 2019-09-23 20:57, Jia Huang wrote: Hi Erik, Thank you for your review and valuable comments. Updated: http://cr.openjdk.java.net/~jiefu/8231351-huangjia/webrev.01/  - The reference to the pkcs11 README and the reviewers had been added. Please note the user in the patc

Re: RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-23 Thread Jia Huang
Hi John Jiang, Thank you for your review. 在 2019年09月23日 20:54, sha.ji...@oracle.com 写道: In fact, PKCS11 tests have their own doc at: test/jdk/sun/security/pkcs11/README I'm afraid most people wouldn't see test/jdk/sun/security/pkcs11/README at all. So it makes very little sense to add the no

Re: RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-23 Thread Jia Huang
Hi Erik, Thank you for your review and valuable comments. Updated: http://cr.openjdk.java.net/~jiefu/8231351-huangjia/webrev.01/ - The reference to the pkcs11 README and the reviewers had been added. Please note the user in the patch. Hope you wouldn't mind it. Thanks. Could you please sponso

Re: RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-23 Thread Erik Joelsson
I think this type of comment fits well in the top level test doc. It just provides basic instructions for setting up these tests so that they pass without going into too much detail. Perhaps a reference to the pkcs11 README for more details would be a good idea. Looks good to me. /Erik On 20

Re: RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-23 Thread sha . jiang
Hi Jia, I think this isn't a general testing problem. It may not worthy of highlighting this point in the JDK testing doc. In fact, PKCS11 tests have their own doc at: test/jdk/sun/security/pkcs11/README Best regards, John Jiang On 2019/9/23 18:04, Jia Huang wrote: Hi all, JBS:    https://bu

RFR: 8231351: Add notes for PKCS11 tests in the test doc

2019-09-23 Thread Jia Huang
Hi all, JBS:https://bugs.openjdk.java.net/browse/JDK-8231351 Webrev: http://cr.openjdk.java.net/~jiefu/8231351-huangjia/webrev.00/ sun/security/pkcs11/Secmod/AddTrustedCert.java failed on Ubuntu 18.04. According to the comments in JDK-8231338, it was caused by the improper NSS libs of the s