Thank you Julia!
Looks good to me.

Thanks,
Amy

On 3/24/20 11:01 PM, Julia Boes wrote:
Hi Amy,

Just a minor comment on the test usage of

/othervm/java.security.policy=FilePublisherPermsTest3.policy -Djava.security.manager

As jtreg tag-spec [1] suggested:
  The setting of the system properties "java.security.manager" and "java.security.policy" in main/othervm actions is also discouraged; the /policy and /secure options should be used instead.

thanks for spotting that! It's fixed now.

Updated webrev: http://cr.openjdk.java.net/~jboes/webrevs/8235459/webrev.03/

Cheers,

Julia



Reply via email to