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
