On Sun, Oct 20, 2013 at 06:48:31PM +0800, Ming Liu wrote:
> In some cases, it's unfit to use "+=" in a conditional appending, we would
> end up with the variable being set rather than being appended, which is not
> it mean to.
>
> Signed-off-by: Ming Liu <ming....@windriver.com>
> ---
> meta/recipes-devtools/strace/strace_4.8.bb | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-devtools/strace/strace_4.8.bb
> b/meta/recipes-devtools/strace/strace_4.8.bb
> index 79a4ad7..aad53b7 100644
> --- a/meta/recipes-devtools/strace/strace_4.8.bb
> +++ b/meta/recipes-devtools/strace/strace_4.8.bb
> @@ -19,7 +19,7 @@ inherit autotools ptest
> RDEPENDS_${PN}-ptest += "make"
>
> PACKAGECONFIG_class-target ?= "libaio"
> -PACKAGECONFIG_class-target += "${@base_contains('DISTRO_FEATURES', 'acl',
> 'acl', '', d)}"
> +PACKAGECONFIG_append_class-target = " ${@base_contains('DISTRO_FEATURES',
> 'acl', 'acl', '', d)}"
Doesn't using _append make it unnecessary harder to changed default value
in .bbappend?
Why not use
PACKAGECONFIG_class-target ?= "libaio ${@base_contains('DISTRO_FEATURES',
'acl', 'acl', '', d)}"
at least in this case?
Thanks
--
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com
signature.asc
Description: Digital signature
_______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
