RFR 8184904/10, jdk/internal/jrtfs/WithSecurityManager fails with exploded builds

[Felix Yang]

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
>
>