Looks fine to me. Thanks, Xuelei
On 6/7/2016 9:51 AM, Wang Weijun wrote: > I forgot to list (enum Option) DESTPROTECTED as an option of (enum Command) > IMPORTKEYSTORE. Here is the fix. Please take a review: > > diff --git a/src/java.base/share/classes/sun/security/tools/keytool/Main.java > b/src/java.base/share/classes/sun/security/tools/keytool/Main.java > --- a/src/java.base/share/classes/sun/security/tools/keytool/Main.java > +++ b/src/java.base/share/classes/sun/security/tools/keytool/Main.java > @@ -206,7 +206,7 @@ > IMPORTKEYSTORE("Imports.one.or.all.entries.from.another.keystore", > SRCKEYSTORE, DESTKEYSTORE, SRCSTORETYPE, > DESTSTORETYPE, SRCSTOREPASS, DESTSTOREPASS, > - SRCPROTECTED, SRCPROVIDERNAME, DESTPROVIDERNAME, > + SRCPROTECTED, DESTPROTECTED, SRCPROVIDERNAME, DESTPROVIDERNAME, > SRCALIAS, DESTALIAS, SRCKEYPASS, DESTKEYPASS, > NOPROMPT, PROVIDERCLASS, PROVIDERARG, PROVIDERPATH, > V), > > Will be labeled noreg-trivial. > > Thanks > Max >