Thanks, Roger. Cheers, Henry
> On Mar 1, 2019, at 10:27 AM, Roger Riggs <roger.ri...@oracle.com> wrote: > > Hi Henry, > > The change looks ok, Reviewed. > > (I'm not that familiar with Solaris). > > Thanks, Roger > > > > On 3/1/19 10:45 AM, Henry Jen wrote: >> Ping! >> >> Any thought on this webrev restore the way for Solaris LD_LIBRARY_PATH_64 >> handling? >> >> Cheers, >> Henry >> >> >>> On Feb 13, 2019, at 9:37 AM, Henry Jen <henry....@oracle.com> wrote: >>> >>> Hi, >>> >>> Please review the webrev[1] for 8217216. The fix makes sure on Solaris, >>> when LD_LIBRARY_PATH_64 is set, we setup LD_LIBRARY_PATH based on that >>> value and unset LD_LIBRARY_PATH_64 in the relaunched process. >>> >>> Same approach was used before JDK-6367077, and the override is expected >>> behavior on Solaris 64-bit as stated in Solaris 64-bit Developer's Guide[2], >>> "The 64-bit dynamic linker's search path can be completely overridden using >>> the LD_LIBRARY_PATH_64 environment variable." >>> >>> Cheers, >>> Henry >>> >>> [1] http://cr.openjdk.java.net/~henryjen/8217216/0/webrev/ >>> [2] https://docs.oracle.com/cd/E19455-01/806-0477/dev-env-7/index.html >>> >