Re: [PATCH v4 2/2] scripts: generate_rust_analyzer.py: fix IDE support for primitive types
From: Tamir Duberstein
Date: Thu Mar 19 2026 - 11:06:24 EST
On Tue, 17 Mar 2026 18:29:55 +0900, Jesung Yang <y.j3ms.n@xxxxxxxxx> wrote:
> diff --git a/scripts/generate_rust_analyzer.py b/scripts/generate_rust_analyzer.py
> index 21832763c5be..bf5a0f16995a 100755
> --- a/scripts/generate_rust_analyzer.py
> +++ b/scripts/generate_rust_analyzer.py
> @@ -372,6 +374,7 @@ def generate_crates(
> path,
> sysroot_deps(core) + [kernel, pin_init],
> cfg=generated_cfg,
> + crate_attrs=["no_std"],
> )
This seems to be independent of the RA version. Is this broken today? Should it
be a separate patch? Maybe that patch need not be in this series.
> @@ -432,6 +435,18 @@ def generate_rust_project(
> ),
> "sysroot": str(sysroot),
> }
> + elif version_info == RaVersionInfo.SUPPORTS_CRATE_ATTRS:
> + ctx = RaVersionCtx(
I think you forgot to define RaVersionInfo.SUPPORTS_CRATE_ATTRS?
> [ ... skip 7 lines ... ]
> + "sysroot": str(sysroot),
> + "sysroot_src": str(sysroot_src),
> + }
> else:
> assert_never(version_info)
>
This bifurcates the handling of RA version differences: half of it is encoded
in RaVersionCtx, and the other half is here in the presence or absence of
"sysroot_src". Can we keep it somehow more contained?
--
Tamir Duberstein <tamird@xxxxxxxxxx>