Hi, Fix looks good, but maybe we should have a regression test of GetJREPath()?
/Magnus > 5 nov. 2018 kl. 12:09 skrev Priyanka Mangal <priyanka.man...@oracle.com>: > > Hi, > > Please review the minor fix for : > https://bugs.openjdk.java.net/browse/JDK-8213362 > http://cr.openjdk.java.net/~pmangal/8213362/webrev.00/ > > With the JDK-8210931 <https://bugs.openjdk.java.net/browse/JDK-8210931> > libjli has moved from "jli" subdirectory into standard lib directory. > However `GetJREPath` method in > `src/java.base/macosx/native/libjli/java_md_macosx.m` still assumes that > `libjli.dylib` should be on `lib/jli/libjli.dylib` path. > So corrected the same. > > Testing: > mach5 : tier1-3 > http://java.se.oracle.com:10065/mdash/jobs/fmatte-jdk-20181105-0941-8721?search=result.status:* > > Thanks > Regards > Priyanka > >
