Addressed all comments. Please review the updated patch:
http://cr.openjdk.java.net/~xiaofeya/8181080/webrev.01/
Thanks,
Felix
On 2017/6/6 16:25, Alan Bateman wrote:
On 06/06/2017 09:15, Chris Hegarty wrote:
:
OtherResourcesTest.java is not strictly needed. You should be able to
provide an appropriate @run tag to OtherResources.java directly, like:
@run main/othervm --limit-modules=java.base OtherResources
As I recall, the reason we created this test a shell test, is because
jtreg couldn't handle --limit-modules in the @run tag. So I agree we
should go back to it and ideally just specify it to the @run tag as
per your mail.
-Alan
