> -----Original Message-----
> From: openembedded-core@lists.openembedded.org 
> <openembedded-core@lists.openembedded.org> On Behalf Of K Sanjay Nayak
> Sent: den 12 april 2024 08:06
> To: openembedded-core@lists.openembedded.org
> Cc: K Sanjay Nayak <nayakksan...@gmail.com>
> Subject: [OE-core] [PATCH] Replaced :append with += in multiple recipe files 
> under meta layer.
> 
> Fixes [YOCTO #2951]
> 
> Conditional operator is preferred over the append operation,
> hence modified multiple recipes under the meta-layer.
> 
> Reported-by : Christopher Larson <kerg...@gmail.com>
> Signed-off-by: K Sanjay Nayak <nayakksan...@gmail.com>
> ---
>  meta/recipes-bsp/grub/grub-efi_2.12.bb                          | 2 +-
>  meta/recipes-core/initscripts/initscripts_1.0.bb                | 2 +-
>  meta/recipes-core/sysvinit/sysvinit_3.04.bb                     | 2 +-
>  .../baremetal-example/baremetal-helloworld_git.bb               | 2 +-
>  meta/recipes-extended/less/less_643.bb                          | 2 +-
>  meta/recipes-gnome/librsvg/librsvg_2.57.1.bb                    | 2 +-
>  meta/recipes-graphics/waffle/waffle_1.7.2.bb                    | 2 +-
>  meta/recipes-support/libgpg-error/libgpg-error_1.48.bb          | 2 +-
>  meta/recipes-support/ptest-runner/ptest-runner_2.4.3.bb         | 2 +-
>  9 files changed, 9 insertions(+), 9 deletions(-)
> 

[cut]

> diff --git 
> a/meta/recipes-extended/baremetal-example/baremetal-helloworld_git.bb 
> b/meta/recipes-extended/baremetal-example/baremetal-helloworld_git.bb
> index 6832ccc541..f66581cca3 100644
> --- a/meta/recipes-extended/baremetal-example/baremetal-helloworld_git.bb
> +++ b/meta/recipes-extended/baremetal-example/baremetal-helloworld_git.bb
> @@ -25,7 +25,7 @@ inherit baremetal-image
> 
> 
>  # startup code for x86 uses NASM syntax
> -DEPENDS:qemux86:append = " nasm-native"
> +DEPENDS:qemux86 += " nasm-native"

While there may be cases where the above may be correct, I do 
not think this is one of them, especially as this is an example 
recipe. The above should be:

DEPENDS:append:qemux86 = " nasm-native"

//Peter

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#198172): 
https://lists.openembedded.org/g/openembedded-core/message/198172
Mute This Topic: https://lists.openembedded.org/mt/105479729/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to