On Wed, 2022-12-21 at 12:53 +0000, Alex Kiernan wrote:
> Use SDK_TOOLCHAIN_LANGS to allow different language support to be
> selected within SDKs. Initially supported options are rust and go.
> 
> Signed-off-by: Alex Kiernan <[email protected]>
> ---
> 
> (no changes since v1)
> 
>  .../packagegroups/packagegroup-core-sdk.bb             | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb 
> b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
> index d70aff22c725..0e480e6ac756 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-core-sdk.bb
> @@ -11,6 +11,11 @@ inherit packagegroup
>  
>  #PACKAGEFUNCS =+ 'generate_sdk_pkgs'
>  
> +SDK_TOOLCHAIN_LANGS ??= ""
> +SDK_TOOLCHAIN_LANGS:remove:sdkmingw32 = "rust"
> +# libstd-rs doesn't build for mips n32 with compiler constraint errors
> +SDK_TOOLCHAIN_LANGS:remove:mipsarchn32 = "rust"
> +
>  RDEPENDS:packagegroup-core-sdk = "\
>      packagegroup-core-buildessential \
>      coreutils \
> @@ -23,7 +28,10 @@ RDEPENDS:packagegroup-core-sdk = "\
>      less \
>      ldd \
>      file \
> -    tcl"
> +    tcl \
> +    ${@bb.utils.contains('SDK_TOOLCHAIN_LANGS', 'go', 
> 'packagegroup-go-sdk-target', '', d)} \
> +    ${@bb.utils.contains('SDK_TOOLCHAIN_LANGS', 'rust', 
> 'packagegroup-rust-sdk-target', '', d)} \
> +"

I'm a little torn on this. I created SDK_TOOLCHAIN_LANGS as a control
for the SDKs from populate_sdk. Whether the target SDK images should
follow that as well, I'm less sure.

It is probably fine and I nearly didn't send a comment but I just
wanted to see what others think.

We could perhaps do something like:

TARGET_TOOLCHAIN_LANGS ??= "${SDK_TOOLCHAIN_LANGS}"

in the recipe to cover the ability to separate the two if we wanted...

Cheers,

Richard



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#174922): 
https://lists.openembedded.org/g/openembedded-core/message/174922
Mute This Topic: https://lists.openembedded.org/mt/95805446/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to