[1/2] tcmode-default: eliminate needless rust, cargo defaults

From: Randy MacLeod <Randy.MacLeod@windriver.com>

Manually setting preferred version is not required so remove
that for rust and cargo.

Signed-off-by: Randy MacLeod <Randy.MacLeod@windriver.com>
---
 meta/conf/distro/include/tcmode-default.inc | 13 -------------
 1 file changed, 13 deletions(-)

I know you want to eliminate this whole file, but I find RUSTVERSION
useful in situations where some layer wants to backport different
version of rust and related recipes and then the DISTRO can set just
the RUSTVERSION variable to choose between the versions available. Now
the layer with different rust recipes will probably need to provide
some .inc file with this, so that the users of this layer won't need
to know which 8 recipes need to have PREFERRED_VERSION set at the same
time.

Like it was with
https://github.com/meta-rust/meta-rust/commits/master/conf/distro/include/rust_versions.inc
https://github.com/meta-rust/meta-rust/commit/9c0a1059be62417439615126ddd3044269406a66

otherwise the version from layer with higher priority will win and the
DISTRO won't be able to easily avoid that, other than adding all these
PREFERRED_VERSIONS or using BBMASK or preventing that layer to be
included in BBLAYERS.

Just my 2c

On Tue, Mar 11, 2025 at 5:25 PM Randy MacLeod via
lists.openembedded.org
<randy.macleod=windriver.com@lists.openembedded.org> wrote:
>
> From: Randy MacLeod <Randy.MacLeod@windriver.com>
>
> Manually setting preferred version is not required so remove
> that for rust and cargo.
>
> Signed-off-by: Randy MacLeod <Randy.MacLeod@windriver.com>
> ---
>  meta/conf/distro/include/tcmode-default.inc | 13 -------------
>  1 file changed, 13 deletions(-)
>
> diff --git a/meta/conf/distro/include/tcmode-default.inc b/meta/conf/distro/include/tcmode-default.inc
> index 702fa4d75f..5cc93405f1 100644
> --- a/meta/conf/distro/include/tcmode-default.inc
> +++ b/meta/conf/distro/include/tcmode-default.inc
> @@ -14,7 +14,6 @@ SDKGCCVERSION ?= "${GCCVERSION}"
>  BINUVERSION ?= "2.44%"
>  GLIBCVERSION ?= "2.41%"
>  LINUXLIBCVERSION ?= "6.12%"
> -RUSTVERSION ?= "1.83%"
>
>  PREFERRED_VERSION_gcc ?= "${GCCVERSION}"
>  PREFERRED_VERSION_gcc-cross-${TARGET_ARCH} ?= "${GCCVERSION}"
> @@ -44,15 +43,3 @@ PREFERRED_VERSION_glibc-mtrace             ?= "${GLIBCVERSION}"
>  PREFERRED_VERSION_glibc-scripts            ?= "${GLIBCVERSION}"
>  PREFERRED_VERSION_nativesdk-glibc          ?= "${GLIBCVERSION}"
>  PREFERRED_VERSION_cross-localedef-native   ?= "${GLIBCVERSION}"
> -
> -# Rust toolchain preferred versions:
> -
> -PREFERRED_VERSION_cargo ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_cargo-native ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_libstd-rs ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_rust ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_rust-cross-${TARGET_ARCH} ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_rust-llvm ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_rust-llvm-native ?= "${RUSTVERSION}"
> -PREFERRED_VERSION_rust-native ?= "${RUSTVERSION}"
> -
> --
> 2.34.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#212574): https://lists.openembedded.org/g/openembedded-core/message/212574
> Mute This Topic: https://lists.openembedded.org/mt/111643200/3617156
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [martin.jansa@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>

On Tue, 2025-03-11 at 18:06 +0100, Martin Jansa via lists.openembedded.org wrote:
> I know you want to eliminate this whole file, but I find RUSTVERSION
> useful in situations where some layer wants to backport different
> version of rust and related recipes and then the DISTRO can set just
> the RUSTVERSION variable to choose between the versions available. Now
> the layer with different rust recipes will probably need to provide
> some .inc file with this, so that the users of this layer won't need
> to know which 8 recipes need to have PREFERRED_VERSION set at the same
> time.
> 
> Like it was with
> https://github.com/meta-rust/meta-rust/commits/master/conf/distro/include/rust_versions.inc
> https://github.com/meta-rust/meta-rust/commit/9c0a1059be62417439615126ddd3044269406a66
> 
> otherwise the version from layer with higher priority will win and the
> DISTRO won't be able to easily avoid that, other than adding all these
> PREFERRED_VERSIONS or using BBMASK or preventing that layer to be
> included in BBLAYERS.
> 
> Just my 2c

In some ways these patches are to check who is actually using some of
these things. They need maintaining and I've not been convinced some of
them are used.

If you are saying you find the rust version useful, that is probably a
reason to keep this, annoying as it can be to maintain...

Cheers,

Richard