It looks good to me. Xuelei
On 5/8/2019 7:58 AM, Weijun Wang wrote:
This test timed out on solaris-sparc multiple times while generating an 8192-bit RSA key pair. Since the test is used to ensure keytool's support for the RSASSA-PSS algorithm (and the default signature algorithm depending on the key size) it's not platform related. Therefore I suggest simply ignoring the test on Solaris. Please take a review on the patch below: diff --git a/test/jdk/sun/security/tools/keytool/PSS.java b/test/jdk/sun/security/tools/keytool/PSS.java --- a/test/jdk/sun/security/tools/keytool/PSS.java +++ b/test/jdk/sun/security/tools/keytool/PSS.java @@ -23,11 +23,12 @@ /* * @test - * @bug 8215694 + * @bug 8215694 8222987 * @summary keytool cannot generate RSASSA-PSS certificates * @library /test/lib * @modules java.base/sun.security.util * java.base/sun.security.x509 + * @requires os.family != "solaris" * @run main PSS */ Thanks, Max