Re: [OE-core] [PATCH V2 1/2] rust: fix for rust multilib sdk configuration

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Thu, 2025-01-16 at 05:48 -0800, Sadineni, Harish via lists.openembedded.org wrote:
> From: Harish Sadineni <Harish.Sadineni@windriver.com>
> 
> YOCTO [#15061]
> The rust sdk installs both 'rust.sh' and 'cargo.sh' for lib32 and lib64 in the same location.
> This causes below error while installing the lib32 & lib64 binaries:
> 
> Error: Transaction test error:
>   file /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/environment-setup.d/cargo.sh
> conflicts between attempted installs of rust-cross-canadian-arm-1.67.1-r0.x86_64_nativesdk and
>  rust-cross-canadian-aarch64-1.67.1-r0.x86_64_nativesdk
>   file /usr/local/oe-sdk-hardcoded-buildpath/sysroots/x86_64-pokysdk-linux/environment-setup.d/rust.sh
> conflicts between attempted installs of rust-cross-canadian-arm-1.67.1-r0.x86_64_nativesdk and
> rust-cross-canadian-aarch64-1.67.1-r0.x86_64_nativesdk
> ERROR: Task (virtual:multilib:lib32:/media/build/poky/meta/recipes-sato/images/core-image-sato.bb:do_populate_sdk)
> failed with exit code '1'
> 
> The change includes:
> - Prepending '${RUST_TARGET_SYS}' to 'rust.sh' to differentiate between target systems.
> - Moving the non-target-specific environment variables to 'nativesdk-cargo' and 'nativesdk-rust',
> instead of being managed by the cross-canadian recipe.
> - Removed the redundant cargo config file from the 'rust-cross-canadian.inc' file.
> 
> Signed-off-by: Harish Sadineni <Harish.Sadineni@windriver.com>
> ---
>  meta/recipes-devtools/rust/cargo_1.81.0.bb    | 15 +++++++++++
>  .../rust/rust-cross-canadian.inc              | 27 ++-----------------
>  meta/recipes-devtools/rust/rust_1.81.0.bb     | 13 +++++++++
>  3 files changed, 30 insertions(+), 25 deletions(-)

Thanks for the patch. Just to confirm, are you saying we no longer need
the $CARGO_HOME/config file?

Cheers,

Richard