Yes, you are right. Webrev updated at http://cr.openjdk.java.net/~weijun/6997010/webrev.02. GendataJavaSecurity.gmk and MakeJavaSecurity.java updated.
Thanks Max On Jul 28, 2014, at 19:43, Erik Joelsson <erik.joels...@oracle.com> wrote: > Hello Max, > > Shouldn't the rule for $(GENDATA_JAVA_SECURITY) depend on > $(RESTRICTED_PKGS_SRC) so that updates to the pkgs file triggers a rebuild? > For that to work, the variable $(RESTRICTED_PKGS_SRC) needs to be empty for > the OPENJDK case rather than have a dummy name and MakeJavaSecurity.java > needs to handle missing the last argument. > > /Erik > > On 2014-07-28 03:44, Wang Weijun wrote: >> 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 >>> >>> >