RFR: 8164071: Default.policy file missing content for solaris

[Erik Joelsson]

Hello Sean,

The change looks ok, but it could also be expressed like this to avoid 
duplication:

ifneq ($(filter $(OPENJDK_TARGET_OS), windows solaris), )
   DEF_POLICY_SRC_LIST += $(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/lib/security/default.policy
endif


/Erik

On 2016-08-17 18:18, Sean Mullan wrote:
> Please review this simple fix to append the solaris default.policy 
> file to the overall default.policy file.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8164071
>
> diff -r 551f7617b2c0 make/copy/Copy-java.base.gmk
> --- a/make/copy/Copy-java.base.gmk    Wed Aug 17 10:08:18 2016 +0800
> +++ b/make/copy/Copy-java.base.gmk    Wed Aug 17 12:17:19 2016 -0400
> @@ -185,6 +185,8 @@
>
>  ifeq ($(OPENJDK_TARGET_OS), windows)
>    DEF_POLICY_SRC_LIST += 
> $(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/lib/security/default.policy
> +else ifeq ($(OPENJDK_TARGET_OS), solaris)
> +  DEF_POLICY_SRC_LIST += 
> $(JDK_TOPDIR)/src/java.base/$(OPENJDK_TARGET_OS)/lib/security/default.policy 
>
>  endif
>
>  # Allow imported modules to modify the java.policy
>
>
> Thanks,
> Sean