Looks fine.

--Sean

On 06/17/2016 08:40 AM, Wang Weijun wrote:
I forgot to update the test as well. Please review this patch:

diff --git a/test/sun/security/tools/jarsigner/warnings/Test.java 
b/test/sun/security/tools/jarsigner/warnings/Test.java
--- a/test/sun/security/tools/jarsigner/warnings/Test.java
+++ b/test/sun/security/tools/jarsigner/warnings/Test.java
@@ -121,10 +121,10 @@
              + "(%1$tY-%1$tm-%1$td).";

      static final String NO_TIMESTAMP_VERIFYING_WARN_TEMPLATE
-            = "This jar contains signatures that does not include a timestamp. 
"
+            = "This jar contains signatures that do not include a timestamp. "
              + "Without a timestamp, users may not be able to validate this jar 
"
-            + "after the signer certificate's expiration date "
-            + "(%1$tY-%1$tm-%1$td).";
+            + "after any of the signer certificates expire "
+            + "(as early as %1$tY-%1$tm-%1$td).";

      static final String NOT_YET_VALID_CERT_SIGNING_WARNING
              = "The signer certificate is not yet valid.";

Thanks
Max

Reply via email to