Thanks for the review!
Valerie

On 10/24/2014 3:29 PM, Mandy Chung wrote:

On 10/24/2014 3:06 PM, Valerie Peng wrote:

Can someone help reviewing this fix? This is just to add a custom header for native Ucrypto functions so that official Solaris build using S10u6 devkit can continue to work after 8046002 is integrated.

Webrev: http://cr.openjdk.java.net/~valeriep/8061574/webrev.00/

Both local build (with the --with-devkit=/java/devtools/i586/devkit/SS12u3-Solaris10u6 configuration option) as well as JPRT job (-buildsonly) succeeded.

This looks okay to me. I will count on your JPRT, local build testing, and any other relevant build verification that code review can't catch.

Mandy

Reply via email to