RFR 8039898: sunpkcs11-solaris.cfg should be in solaris specific directory

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Thu Sep 11 09:38:07 UTC 2014


On 2014-09-10 18:28, Valerie Peng wrote:
>
> Could someone please review this build related change for moving 
> sunpkcs11-solaris.cfg file to the pkcs11 module?
>
> Webrev: http://cr.openjdk.java.net/~valeriep/8039898/webrev.00/

Looks good. However, I'll give the same feedback to you as I just 
recently did to Phil:

The webrev shows the file as being moved outside the control of 
mercurial. That is, if you do "hg mv" to move the file, the history of 
the file will be kept intact. Otherwise it will look like a new file in 
the repo. (Sometimes this doesn't show up properly in the webrev, 
apologies if you already did this.)

/Magnus



More information about the security-dev mailing list