Re: [PATCH v2 1/2] rust: add untrusted data abstraction
From: Miguel Ojeda
Date: Fri Sep 27 2024 - 04:40:07 EST
On Fri, Sep 27, 2024 at 12:15 AM Benno Lossin <benno.lossin@xxxxxxxxx> wrote:
>
> Personally, I would prefer adding a symbol. Since if we allow it, then
> it might take a long time until the code is removed once we increase the
> minimum version.
>
> It would be best, if it produces an error when we raise the minimum
> version beyond the one represented by the symbol. Is that already the
> case?
No -- that is a nice idea that I guess could be implemented in Kconfig
somewhere (i.e. when checking conditions). However, one of the common
things to do when upgrading the minimum is to review/clean the
`*_VERSION` uses anyway (and they may occur outside Kconfig files
too), and also sometimes one wants to upgrade a minimum without doing
the cleanups immediately (e.g. the recently proposed GCC 8.1 upgrade).
> Gave it a try and I also can't reproduce the error there...
Hmm... I think CE uses the Rust-provided binaries (and I guess the
playground too). Do you have a link ("Share" in CE)?
Cheers,
Miguel