Great. That looks good. Thanks Max
On May 29, 2014, at 22:50, Vincent Ryan <vincent.x.r...@oracle.com> wrote: > Updated webrev at: > http://cr.openjdk.java.net/~vinnie/8044038/webrev.01/ > > > On 29 May 2014, at 14:47, Vincent Ryan <vincent.x.r...@oracle.com> wrote: > >> Sure. I’ll add it to autotest.sh too. >> >> On 29 May 2014, at 14:40, Wang Weijun <weijun.w...@oracle.com> wrote: >> >>> Vinnie >>> >>> The bug report shows sun/security/tools/keytool/autotest.sh also failed. >>> The test includes >>> >>> LIBNAME=`find_one \ >>> "/usr/lib/libsoftokn3.so" \ >>> "/usr/lib/i386-linux-gnu/nss/libsoftokn3.so" \ >>> "/usr/lib/nss/libsoftokn3.so"` >>> >>> Maybe adding a line for /usr/lib32? >>> >>> Have you verified the fix on the machine? >>> >>> Thanks >>> Max >>> >>> On May 29, 2014, at 21:29, Vincent Ryan <vincent.x.r...@oracle.com> wrote: >>> >>>> Please review this trivial change to enable the PKCS11/NSS tests to be run >>>> on older (Ubuntu) Linux 32-bit releases. >>>> The ‘/usr/lib32’ directory is now included in the search order for NSS >>>> libraries. >>>> >>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8044038 >>>> Webrev: http://cr.openjdk.java.net/~vinnie/8044038/webrev.00 >>>> >>>> Thanks. >>>> >>>> >>>> >>> >> >