[9] RFR 8130665 "java/lang/SecurityManager/CheckSecurityProvider.java failing in jake on OS X"

[Erik Joelsson]

Hello Valerie,

To reduce duplication, I think something like this would be preferable. 
Remove the windows and macosx checks and just always run this line:

POLICY_SRC_LIST += \
     $(wildcard 
$(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/conf/security/java.policy)

That way, I platform specific file is always picked up if found.

/Erik

On 2015-08-08 01:25, Valerie Peng wrote:
> Hi, Sean and build experts,
>
> Can you please review the fix for 8130665 
> "java/lang/SecurityManager/CheckSecurityProvider.java failing in jake 
> on OS X"?
> The Apple provider fails to be instantiated in Jake due to the missing 
> permission grants.
> However, it currently only fails in Jake workspace and fix is verified 
> by running JPRT against Jake.
>
> Given that Apple provider only presents for Mac OSX, its policy file 
> is under the macosx directory and I have to make minor modifications 
> to pick up that file for MacOSX build.
>
> Here is the webrev:
> http://cr.openjdk.java.net/~valeriep/8130665/webrev.00/
>
> Thanks,
> Valerie