On 20/07/2017 04:22, Felix Yang wrote:
Hi Alan,
if I understand correctly, you meant that just to check for
${java.home}/lib/modules is enough?
Please review the updated patch. I also adjusted checking in
Basic.java, which also has duplicate checking and is problematic
(jrt-fs.jar is no longer under JDK root directory)
Webrev: http://cr.openjdk.java.net/~xiaofeya/8184904/webrev.01/
This skips the testing on exploded builds, which I think is okay. A
minor point before pushing is that you replace "lib/modules" with "lib",
"modules" as the elements will be joined by the method.
-Alan
