Hi Hemanth,
On Wed, 2026-06-24 at 23:52 -0700, Hemanth Kumar M D via
lists.openembedded.org wrote:
> From: Hemanth Kumar M D <[email protected]>
>
> Cargo walks from CWD up to the filesystem root merging every
> .cargo/config[.toml] it finds. Any such file above BASE_WORKDIR is
> silently picked up and can override Yocto's linker, registry or
> compiler settings, leading to build failures.
>
> Until cargo provides a proper fix upstream, add a warning so users
> get a clear diagnostic instead of a build error.
>
> Upstream meta-issue: https://github.com/rust-lang/cargo/issues/9769
>
> [YOCTO #15637]
>
> Signed-off-by: Hemanth Kumar M D <[email protected]>
> ---
> meta/classes-global/sanity.bbclass | 33
> ++++++++++++++++++++++++++++++
> 1 file changed, 33 insertions(+)
>
> diff --git a/meta/classes-global/sanity.bbclass b/meta/classes-
> global/sanity.bbclass
> index bdfa7f059d..5c5cf311da 100644
> --- a/meta/classes-global/sanity.bbclass
> +++ b/meta/classes-global/sanity.bbclass
> @@ -854,6 +854,38 @@ def sanity_check_locale(d):
> except locale.Error:
> raise_sanity_error("Your system needs to support the
> en_US.UTF-8 locale.", d)
>
> +def check_cargo_config(d):
> + # Cargo merges .cargo/config[.toml] from every directory between
> CWD and
> + # the filesystem root. Warn for anything found in ancestor
> directories
> + # above BASE_WORKDIR that Cargo would pick up silently.
> + import os
> +
> + base_workdir = d.getVar('BASE_WORKDIR')
> + ancestor = os.path.dirname(base_workdir)
> + found = []
> + last_ancestor = None
> + while True:
> + for name in ('config', 'config.toml'):
> + cfg = os.path.join(ancestor, '.cargo', name)
> + if os.path.exists(cfg):
> + found.append(cfg)
> + last_ancestor = ancestor
> + break
> + parent = os.path.dirname(ancestor)
> + if parent == ancestor:
> + break
> + ancestor = parent
This can easier be implemented using pathlib.Path:
from pathlib import Path
base_workdir = Path(d.getVar('BASE_WORKDIR'))
for path in base_workdir.parents:
for config in (".cargo/config", ".cargo/config.toml"):
if path.joinpath(config).exists():
found.append(path / config)
// Martin
> +
> + if found:
> + bb.warn("Cargo config file(s) found at %s which is/are
> outside the build "
> + "directory. Cargo will silently apply their settings
> during the "
> + "rust/cargo build and can override Yocto's settings
> like linker, "
> + "registry or compiler settings causing build
> failures. You can "
> + "either remove these file(s) or move your build
> directory outside "
> + "of %s to fix this. "
> + "See
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=15637 for more
> details."
> + % (', '.join(found), last_ancestor))
> +
> def check_sanity_everybuild(status, d):
> import os, stat
> # Sanity tests which test the users environment so need to run
> at each build (or are so cheap
> @@ -873,6 +905,7 @@ def check_sanity_everybuild(status, d):
> status.addresult('Bitbake version %s is required and version
> %s was found\n' % (minversion, bb.__version__))
>
> sanity_check_locale(d)
> + check_cargo_config(d)
>
> paths = d.getVar('PATH').split(":")
> if "." in paths or "./" in paths or "" in paths:
> --
> 2.49.0
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#239625):
https://lists.openembedded.org/g/openembedded-core/message/239625
Mute This Topic: https://lists.openembedded.org/mt/119970000/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
