Re: RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki
> On Jan 20, 2017, at 10:22 AM, Anthony Scarpino > wrote: > > Good catch.. that'll teach me for trusting the graphical tool to rename a > directory when I used 'Rename'. > > Also I found Brad's issue as it was a new changeset that just showed up in > that file. > > I updated the webrev, which looks better now: > http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk.01/ +1 Mandy
Re: RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki
On 01/19/2017 11:50 AM, Mandy Chung wrote: On Jan 19, 2017, at 11:39 AM, Anthony Scarpino wrote: Hi, I need a review to rename the jdk.crypto.token to jdk.crypto.cryptoki. This is to change what 8171202 had done to the original jdk.crypto.pkcs11 module. For those not familiar with discussions elsewhere, the term "token" is confusing and unclear as it can mean many things cryptographically. The use of cryptoki is more accurate to the original 'pkcs11' name and still conforming to the new naming standard. http://cr.openjdk.java.net/~ascarpino/8172527/webrev-root/ http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk/ The webrev shows deleted/new files for src/jdk.crypto.token source files rather than rename. Did you use hg rename? Changes in other files looks good. Mandy Good catch.. that'll teach me for trusting the graphical tool to rename a directory when I used 'Rename'. Also I found Brad's issue as it was a new changeset that just showed up in that file. I updated the webrev, which looks better now: http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk.01/ Tony
Re: RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki
Similar problem about the rename. rename or addremove (likely) should work, but hg add/hg remove won't. To check that all references to jdk.crypto.{token,pkcs11} are gone, I ran a test over lunch and applied the webrev patches to a clean workspace. There was a problem applying the jdk webrev via gpatch on Solaris. This may be a bug in gpatch or pilot error, but please check it out. patching file src/jdk.crypto.token/share/classes/sun/security/pkcs11/P11Signature.java Reversed (or previously applied) patch detected! Assume -R? [n] Then at the end of the gpatch, these files were left in the directory. src/jdk.crypto.token/share/classes/sun/security/pkcs11: P11Signature.java P11Signature.java.orig P11Signature.java.rej Otherwise, everything seems to be fine. Brad On 1/19/2017 11:50 AM, Mandy Chung wrote: On Jan 19, 2017, at 11:39 AM, Anthony Scarpino wrote: Hi, I need a review to rename the jdk.crypto.token to jdk.crypto.cryptoki. This is to change what 8171202 had done to the original jdk.crypto.pkcs11 module. For those not familiar with discussions elsewhere, the term "token" is confusing and unclear as it can mean many things cryptographically. The use of cryptoki is more accurate to the original 'pkcs11' name and still conforming to the new naming standard. http://cr.openjdk.java.net/~ascarpino/8172527/webrev-root/ http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk/ The webrev shows deleted/new files for src/jdk.crypto.token source files rather than rename. Did you use hg rename? Changes in other files looks good. Mandy
Re: RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki
> On Jan 19, 2017, at 11:39 AM, Anthony Scarpino > wrote: > > Hi, > > I need a review to rename the jdk.crypto.token to jdk.crypto.cryptoki. This > is to change what 8171202 had done to the original jdk.crypto.pkcs11 module. > For those not familiar with discussions elsewhere, the term "token" is > confusing and unclear as it can mean many things cryptographically. The use > of cryptoki is more accurate to the original 'pkcs11' name and still > conforming to the new naming standard. > > http://cr.openjdk.java.net/~ascarpino/8172527/webrev-root/ > http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk/ The webrev shows deleted/new files for src/jdk.crypto.token source files rather than rename. Did you use hg rename? Changes in other files looks good. Mandy
RFR 8172527: Rename jdk.crypto.token to jdk.crypto.cryptoki
Hi, I need a review to rename the jdk.crypto.token to jdk.crypto.cryptoki. This is to change what 8171202 had done to the original jdk.crypto.pkcs11 module. For those not familiar with discussions elsewhere, the term "token" is confusing and unclear as it can mean many things cryptographically. The use of cryptoki is more accurate to the original 'pkcs11' name and still conforming to the new naming standard. http://cr.openjdk.java.net/~ascarpino/8172527/webrev-root/ http://cr.openjdk.java.net/~ascarpino/8172527/webrev-jdk/ thanks Tony