Re: [dunfell/master][PATCH] ti-test: fix string expansion

[Denys Dmytriyenko <denis@denix.org>]

On Fri, Apr 16, 2021 at 02:30:53PM -0500, Gowtham Tammana wrote:
> bb conditional util expansion was done in '' causing the below error
> 
> ```
> bb.data_smart.ExpansionError: Failure expanding variable ARAGO_TI_TEST, expression was ... ${@bb.utils.contains('MACHINE_FEATURES', 'mmip', 'omapdrmtest', '', d)} cmem-test     ', d)}
> which triggered exception SyntaxError: invalid syntax (Var <ARAGO_TI_TEST>, line 1)
> ```
> 
> fixed the expansion to within "".

Seems to have no issues in master, but yeah, good catch.


> Signed-off-by: Gowtham Tammana <g-tammana@ti.com>

Reviewed-by: Denys Dmytriyenko <denys@konsulko.com>


> ---
>  meta-arago-distro/recipes-core/packagegroups/ti-test.bb | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/ti-test.bb b/meta-arago-distro/recipes-core/packagegroups/ti-test.bb
> index 5b04d85c..cbc246d5 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/ti-test.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/ti-test.bb
> @@ -68,11 +68,11 @@ ARAGO_TI_TEST_append_ti43x = " \
>  
>  ARAGO_TI_TEST_append_omap-a15 = " \
>      omapconf \
> -    ${@oe.utils.conditional('ARAGO_BRAND', 'mainline', '', ' \
> +    ${@oe.utils.conditional('ARAGO_BRAND', 'mainline', '', " \
>          ti-ipc-test \
>          ${@bb.utils.contains('MACHINE_FEATURES', 'mmip', 'omapdrmtest', '', d)} \
>          cmem-test \
> -    ', d)} \
> +    ", d)} \
>  "
>  
>  ARAGO_TI_TEST_append_k3 = " \