[oe] [meta-java][PATCH] openjdk-8-cross.inc: tweak PACKAGECONFIG default value

[Maxin B. John]

Hi,

On Wed, Apr 19, 2017 at 05:59:34AM +0200, liu.ming50 at gmail.com wrote:
> From: Ming Liu <peter.x.liu at external.atlascopco.com>
> 
> When x11, cups, alsa is being set in DISTRO_FEATURES, the users might
> expect openjdk build with the related options as well. For instance,
> they might be confused when the openjdk is still built as headless but
> x11 is actually enabled from system aspect.
> 
> With this patch, the end users still can choose build openjdk without
> these options by overriding PACKAGECONFIG.
>
> Signed-off-by: Ming Liu <peter.x.liu at external.atlascopco.com>
> ---
>  recipes-core/openjdk/openjdk-8-cross.inc | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/recipes-core/openjdk/openjdk-8-cross.inc b/recipes-core/openjdk/openjdk-8-cross.inc
> index 47fe35d..34e1de4 100644
> --- a/recipes-core/openjdk/openjdk-8-cross.inc
> +++ b/recipes-core/openjdk/openjdk-8-cross.inc
> @@ -16,6 +16,13 @@ PRIVATE_LIBS = "\
>  "

Please remove this line:

>  PACKAGECONFIG ??= "repack"
> +PACKAGECONFIG ?? = " \
> +    repack \
> +    ${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'x11', '', d)} \
> +    ${@bb.utils.contains('DISTRO_FEATURES', 'cups', 'cups', '', d)} \

Is "cups" a valid distro feature in any reference distributions ? 

> +    ${@bb.utils.contains('DISTRO_FEATURES', 'alsa', 'alsa', '', d)} \
> +"
> +
>  PACKAGECONFIG[x11] = "--with-x,,libx11 xproto libxt libxext libxrender"
>  PACKAGECONFIG[cups] = "--with-cups,,cups"
>  PACKAGECONFIG[alsa] = "--with-alsa,,alsa-lib-native"

Best Regards,
Maxin