[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fb47fe55-7e9c-4d2b-91d2-1c6f29a77f79@gmail.com>
Date: Fri, 5 Sep 2025 07:15:15 +0800
From: Asuna <spriteovo@...il.com>
To: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
Cc: Conor Dooley <conor@...nel.org>, Jason Montleon <jmontleo@...hat.com>,
Han Gao <rabenda.cn@...il.com>, Miguel Ojeda <ojeda@...nel.org>,
Alex Gaynor <alex.gaynor@...il.com>, Boqun Feng <boqun.feng@...il.com>,
Gary Guo <gary@...yguo.net>, Björn Roy Baron
<bjorn3_gh@...tonmail.com>, Benno Lossin <lossin@...nel.org>,
Andreas Hindborg <a.hindborg@...nel.org>, Alice Ryhl <aliceryhl@...gle.com>,
Trevor Gross <tmgross@...ch.edu>, Danilo Krummrich <dakr@...nel.org>,
Jonathan Corbet <corbet@....net>, Paul Walmsley <paul.walmsley@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>,
Alexandre Ghiti <alex@...ti.fr>, Masahiro Yamada <masahiroy@...nel.org>,
Nathan Chancellor <nathan@...nel.org>,
Nick Desaulniers <nick.desaulniers+lkml@...il.com>,
Bill Wendling <morbo@...gle.com>, Justin Stitt <justinstitt@...gle.com>,
Kees Cook <kees@...nel.org>, Tejun Heo <tj@...nel.org>,
Peter Zijlstra <peterz@...radead.org>, Matthew Maurer <mmaurer@...gle.com>,
Jeff Xu <jeffxu@...omium.org>, Jan Hendrik Farr <kernel@...rr.cc>,
Shakeel Butt <shakeel.butt@...ux.dev>, Michal Koutný
<mkoutny@...e.com>, Christian Brauner <brauner@...nel.org>,
Brian Gerst <brgerst@...il.com>, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-riscv@...ts.infradead.org,
linux-kbuild@...r.kernel.org, llvm@...ts.linux.dev,
rust-for-linux <rust-for-linux@...r.kernel.org>
Subject: Re: [PATCH 1/2] rust: get the version of libclang used by bindgen in
a separate script
On 9/4/25 7:24 AM, Miguel Ojeda wrote:
> Hmm... I am not sure it is a good idea to move that into another
> script. Do we really need to intertwine these two scripts? The rename
> isn't great either.
>
Because of adding a new Kconfig symbol for the Rust bindgen libclang
version, then we have three places manually calling bindgen for
rust_is_available_bindgen_libclang.h to get the version. I'd like to
merge them into one script so that it's easy to maintain in the future.
But if you prefer not to, I'd also be willing to revert it.
For this approach and naming, I referred to script/cc-version.sh
rustc-version.sh and rustc-llvm-version.sh.
Powered by blists - more mailing lists