Date: Wed, 30 Oct 2024 16:16:51 +0100
From: Matthias Geiger <werdahias@riseup.net>
To: Miguel Ojeda <miguel.ojeda.sandonis@gmail.com>,
debian-rust@lists.debian.org
Cc:
Bcc:
Subject: Re: Debian & bindgen issue
User-Agent: meli 0.8.7
References: <CANiq72m7z4ks__oVow7FWvZ3cJERf6E=6f3qHsrXgyQAVbNN3A@mail.gmail.com>
In-Reply-To: <CANiq72m7z4ks__oVow7FWvZ3cJERf6E=6f3qHsrXgyQAVbNN3A@mail.gmail.com>
On Wed, 30 Oct 2024 16:15, Miguel Ojeda <miguel.ojeda.sandonis@gmail.com> wrote:
Hi Matthias, Debian Rust maintainers,
While researching an issue (that we will trigger in the kernel in the
future) with bindgen 0.69.4 (and libclang >= 19.1), solved in 0.69.5
(as well as the newer 0.70.x), I went to check whether Debian would be
affected.
Debian Testing seems OK since libclang is older -- I checked using
Docker `debian:testing` image.
Debian Sid triggers the issue, because it has bindgen 0.66.1 which
uses libclang 19.1.2 (2). I confirmed this using the Docker
`debian:sid` image.
Thanks for looking into this.
Do you plan to backport the fix or to upgrade bindgen there? (as far
as I understand, you already carry patches for other bugs in Debian's
bindgen). Or, perhaps, to provide a newer bindgen in a versioned
package, like the Ubuntu's rust-* versioned packages?
The fix I am referring to is:
https://github.com/rust-lang/rust-bindgen/pull/2824
i.e. these commits in upstream bindgen:
35f0f9aafc9b ("Use clang_getFileLocation instead of
clang_getSpellingLocation")
600f63895f73 ("Use clang_getFileLocation instead of
clang_getSpellingLocation")
Are we talking about the bingen binary ? If so, we should look into
updating to 0.70.x imo. We haven't done so yet (and cherry-picked a lot of
patches) because a bingen update involves a lot of crates to be ported
at the same time. I might upload rust-bindgen-cli and rust-bindgen
0.70.x to experimental this week; but the transition will take time.
Maybe someone from the team wants to help. It's not a high priority for me
though tbh. In the meantime, please file a bug for your issue.
best,
Matthias Geiger <werdahias>