Webrev updated at http://cr.openjdk.java.net/~weijun/6997010/webrev.01/
New test CheckSecurityProvider.java, and updates to MakeJavaSecurity.addPackages(). Thanks Max On Jul 25, 2014, at 22:44, Wang Weijun <weijun.w...@oracle.com> wrote: > > On Jul 25, 2014, at 22:30, Sean Mullan <sean.mul...@oracle.com> wrote: > >>> http://cr.openjdk.java.net/~weijun/6997010/webrev.00/ >>> >>> 4. *IMPORTANT*: In order to easily maintain platform-related entries, >>> every line (including the last line) in package.access and >>> package.definition MUST end with ',\' now. A blank line MUST exist >>> after the last line. This avoid ugly lines like >>> >>> #ifndef windows entry1. #endif #ifdef windows entry1.,\ entry2 >>> #endif >> >> What happens if someone (inevitably) adds a new package to the list and >> forgets to do either of these? Does it result in a build failure? > > No build failure, but > test/java/security/SecurityManager/CheckPackageAccess.java will fail. > > I can add check in the build tool. > >> >> Otherwise looks good, although I think it would be useful to write an >> additional test to make sure the correct providers are installed and ordered >> correctly on the different platforms, something similar to the >> java/lang/SecurityManager/CheckPackageAccess.java test but specific to >> providers. > > OK. > > Thanks > Max >