I now noticed that this was only sent to build-dev. This is not really a build question. Cc:ing core-libs-dev.
/Magnus > 5 nov. 2018 kl. 15:21 skrev Magnus Ihse Bursie > <magnus.ihse.bur...@oracle.com>: > > 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 > >