On Fri, 16 Dec 2022 12:06:14 GMT, Sibabrata Sahoo <ssa...@openjdk.org> wrote:
>> Bill Huang has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request contains five additional >> commits since the last revision: >> >> - Merge branch 'master' into JDK-8295087 >> - Added an extra line to the end of the policy file. >> - AssertThrows an exception in InconsistentEntries test. >> - Refactored to use testng framework for test enviroment setup. >> - Converted security manual tests to automated tests. > > test/jdk/sun/security/provider/PolicyParser/ExtDirs.java line 37: > >> 35: * @summary standard extensions path is hard-coded in default >> 36: * system policy file >> 37: * @run main/othervm/policy=ExtDirs.policy/secure=default ExtDirs > > May be "/secure=default" is optional here. Yes, you are right and we can remove the /secure. According to the JTReg spec. `If the /secure option is not used then the default security manager will be installed.` ------------- PR: https://git.openjdk.org/jdk/pull/10637
