<html>
<head>
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
</head>
<body>
<div class="moz-cite-prefix"><tt>Removed othervm: </tt><tt><a
href="http://cr.openjdk.java.net/~amlu/8239979/webrev.01/index.html">http://cr.openjdk.java.net/~amlu/8239979/webrev.01/index.html</a></tt><tt><br>
</tt><tt><br>
</tt><tt>Thanks,</tt><tt><br>
</tt><tt>Amy</tt><tt><br>
</tt><tt><br>
</tt><tt>On 2/26/20 11:02 AM, Rajan Halade wrote:</tt><tt><br>
</tt></div>
<blockquote type="cite"
cite="mid:DC44816A-964B-4085-B5CA-71E344DF4109@ORACLE.COM">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
<tt>Thanks Amy for the fix! </tt>
<div class=""><tt><br class="">
</tt></div>
<div class=""><tt>Any reason for using othervm to run this test?
Otherwise your fix looks good to me.</tt></div>
<div class=""><tt><br class="">
</tt>
<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><tt>Thanks,</tt></div>
<div><tt>Rajan</tt></div>
</div>
</div>
<div><tt><br class="">
</tt>
<blockquote type="cite" class="">
<div class=""><tt>On Feb 25, 2020, at 6:48 PM, Amy Lu <</tt><tt><a
href="mailto:amy.lu@oracle.com" class=""
moz-do-not-send="true">amy.lu@oracle.com</a></tt><tt>>
wrote:</tt></div>
<tt><br class="Apple-interchange-newline">
</tt>
<div class="">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252" class="">
<div class=""> <tt><span class="">sun/security/tools/keytool/ExtOptionCamelCase.java</span></tt><tt><span
class=""><br class="">
</span></tt><tt><span class=""><br class="">
</span></tt><tt><span class="">This is not a
compile-only test, but due to the missed @run tag,
test is not run.</span></tt><tt><span class=""><br
class="">
</span></tt><tt><span class=""><br class="">
</span></tt><tt><span class="">Please review the patch
to fix this issue.</span></tt><tt><span class=""><br
class="">
</span></tt><tt><span class=""><br class="">
</span></tt><tt><span class="">The patch also fixed
@modules. Test performs deep reflection
setAccessible(true) on a non-public member, `open`
added to the module.</span></tt><tt><span class=""><br
class="">
</span></tt><tt><span class=""><br class="">
</span></tt><tt><span class="">bug: <a
class="moz-txt-link-freetext"
href="https://bugs.openjdk.java.net/browse/JDK-8239979"
moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8239979</a></span></tt><tt><span
class=""><br class="">
</span></tt><tt><span class="">webrev: <a
class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/~amlu/8239979/webrev.00/index.html"
moz-do-not-send="true">http://cr.openjdk.java.net/~amlu/8239979/webrev.00/index.html</a></span></tt><tt><span
class=""><br class="">
</span></tt><tt><span class=""><br class="">
</span></tt><tt><span class="">Thanks,</span></tt><tt><span
class=""><br class="">
</span></tt><tt><span class="">Amy</span></tt><tt><span
class=""><br class="">
<br class="">
</span></tt><tt><br class="">
</tt>
<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><span class=""> </span></tt><tt><span class=""> </span></tt>
</div>
</div>
</blockquote>
</div>
<tt><br class="">
</tt></div>
</blockquote>
<tt><br>
</tt>
</body>
</html>