[meta-virtualization][PATCH] vhost-device-gpio: only enable if meta-clang exists

[meta-virtualization][PATCH] vhost-device-gpio: only enable if meta-clang exists

[Erik Schilling]

This disables the recipe - which depends on clang-native - if meta-clang
is not active.

Link: https://lore.kernel.org/yocto-meta-virtualization/ZNyErpfW8p2lX81Z@nuoska/
Reported-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Suggested-by: Bruce Ashfield <bruce.ashfield@gmail.com>
Signed-off-by: Erik Schilling <erik.schilling@linaro.org>
---
As discussed in the linked thread.

Sorry for the inconveniences!
---
 recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb b/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb
index 4ef9de4..7993116 100644
--- a/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb
+++ b/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb
@@ -10,6 +10,8 @@ DEPENDS += "libgpiod"
 # libgpiod-sys generates bindings using bindgen, which depends on clang
 DEPENDS += "clang-native"
 
+SKIP_RECIPE[vhost-device-gpio] ?= "${@bb.utils.contains('BBFILE_COLLECTIONS', 'clang-layer', '', 'Depends on clang-native from meta-clang which is not included', d)}"
+
 SRC_URI += "crate://crates.io/vhost-device-gpio/0.1.0"
 SRC_URI[vhost-device-gpio-0.1.0.sha256sum] = "f4789dd127ce746d4f702d50256ff09e47b19fdb2bfee88a254b7e48efbf1100"
 

---
base-commit: 28205358690f79e13c1970ea7d61bce0131da027
change-id: 20230816-vhost-device-clang-fix-8348952fd3d8

Best regards,
-- 
Erik Schilling <erik.schilling@linaro.org>

Re: [meta-virtualization][PATCH] vhost-device-gpio: only enable if meta-clang exists

[Bruce Ashfield]

It worked in my testing!

This has now merged.

Bruce

In message: [meta-virtualization][PATCH] vhost-device-gpio: only enable if meta-clang exists
on 16/08/2023 Erik Schilling wrote:

> This disables the recipe - which depends on clang-native - if meta-clang
> is not active.
> 
> Link: https://lore.kernel.org/yocto-meta-virtualization/ZNyErpfW8p2lX81Z@nuoska/
> Reported-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> Suggested-by: Bruce Ashfield <bruce.ashfield@gmail.com>
> Signed-off-by: Erik Schilling <erik.schilling@linaro.org>
> ---
> As discussed in the linked thread.
> 
> Sorry for the inconveniences!
> ---
>  recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb b/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb
> index 4ef9de4..7993116 100644
> --- a/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb
> +++ b/recipes-extended/vhost-device/vhost-device-gpio_0.1.0.bb
> @@ -10,6 +10,8 @@ DEPENDS += "libgpiod"
>  # libgpiod-sys generates bindings using bindgen, which depends on clang
>  DEPENDS += "clang-native"
>  
> +SKIP_RECIPE[vhost-device-gpio] ?= "${@bb.utils.contains('BBFILE_COLLECTIONS', 'clang-layer', '', 'Depends on clang-native from meta-clang which is not included', d)}"
> +
>  SRC_URI += "crate://crates.io/vhost-device-gpio/0.1.0"
>  SRC_URI[vhost-device-gpio-0.1.0.sha256sum] = "f4789dd127ce746d4f702d50256ff09e47b19fdb2bfee88a254b7e48efbf1100"
>  
> 
> ---
> base-commit: 28205358690f79e13c1970ea7d61bce0131da027
> change-id: 20230816-vhost-device-clang-fix-8348952fd3d8
> 
> Best regards,
> -- 
> Erik Schilling <erik.schilling@linaro.org>
> 

Re: [meta-virtualization][PATCH] vhost-device-gpio: only enable if meta-clang exists

[Richard Purdie]

On Wed, 2023-08-16 at 21:03 +0200, Erik Schilling wrote:
> This disables the recipe - which depends on clang-native - if meta-clang
> is not active.
> 
> Link: https://lore.kernel.org/yocto-meta-virtualization/ZNyErpfW8p2lX81Z@nuoska/
> Reported-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> Suggested-by: Bruce Ashfield <bruce.ashfield@gmail.com>
> Signed-off-by: Erik Schilling <erik.schilling@linaro.org>
> ---
> As discussed in the linked thread.
> 
> Sorry for the inconveniences!

Thanks, I'm happy to see this quickly fixed! :)

Cheers,

Richard