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/
Thanks,
Felix
On 2017/7/19 17:50, Alan Bateman wrote:
On 19/07/2017 10:43, Felix Yang wrote:
Hi all,
please review a patch to skip parts of testing with exploded builds.
Bug:
https://bugs.openjdk.java.net/browse/JDK-8184904
Webrev:
http://cr.openjdk.java.net/~xiaofeya/8184904/webrev.00/
The check for the image type should be independent of the allow/deny
parameter. Alternatively, and as jrtfs does support exploded builds,
is to use a different policy file when on an exploded build.
Also checking for an image doesn't only need to check for
${java.home}/lib/modules.
-Alan
