<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Thanks Amy for the fix! <div class=""><br class=""></div><div class="">Any reason for using othervm to run this test? Otherwise your fix looks good to me.</div><div class=""><br class=""><div class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div>Thanks,</div><div>Rajan</div></div>
</div>
<div><br class=""><blockquote type="cite" class=""><div class="">On Feb 25, 2020, at 6:48 PM, Amy Lu <<a href="mailto:amy.lu@oracle.com" class="">amy.lu@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
<meta http-equiv="content-type" content="text/html; charset=UTF-8" class="">
<div class="">
<tt class="">sun/security/tools/keytool/ExtOptionCamelCase.java</tt><tt class=""><br class="">
</tt><tt class=""><br class="">
</tt><tt class="">This is not a compile-only test, but due to the missed @run
tag, test is not run.</tt><tt class=""><br class="">
</tt><tt class=""><br class="">
</tt><tt class="">Please review the patch to fix this issue.</tt><tt class=""><br class="">
</tt><tt class=""><br class="">
</tt><tt class="">The patch also fixed @modules. Test performs deep
reflection setAccessible(true) on a non-public member, `open`
added to the module.</tt><tt class=""><br class="">
</tt><tt class=""><br class="">
</tt><tt class="">bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8239979">https://bugs.openjdk.java.net/browse/JDK-8239979</a></tt><tt class=""><br class="">
</tt><tt class="">webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~amlu/8239979/webrev.00/index.html">http://cr.openjdk.java.net/~amlu/8239979/webrev.00/index.html</a></tt><tt class=""><br class="">
</tt><tt class=""><br class="">
</tt><tt class="">Thanks,</tt><tt class=""><br class="">
</tt><tt class="">Amy</tt><tt class=""><br class="">
<br class="">
</tt><br class="">
<pre style="font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; overflow-wrap: break-word; white-space: pre-wrap;" class="">--- old/test/jdk/sun/security/tools/keytool/ExtOptionCamelCase.java 2020-02-26 10:36:19.000000000 +0800
+++ new/test/jdk/sun/security/tools/keytool/ExtOptionCamelCase.java 2020-02-26 10:36:18.000000000 +0800
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2019, 2020, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -24,11 +24,13 @@
/*
* @test
* @bug 8231950
+ * @summary keytool -ext camel-case shorthand not working
* @modules java.base/sun.security.tools.keytool
+ * java.base/sun.security.tools.keytool:open
* java.base/sun.security.util
* java.base/sun.security.x509
* @compile -XDignore.symbol.file ExtOptionCamelCase.java
- * @summary keytool -ext camel-case shorthand not working
+ * @run main/othervm ExtOptionCamelCase
*/
import sun.security.tools.keytool.Main;</pre>
<tt class=""> </tt><tt class=""> </tt>
</div>
</div></blockquote></div><br class=""></div></body></html>