Control: reassign -1 librust-bindgen-dev
Control: affects -1 + bindgen
Hi Miguel,
Thanks for the very detailed bug report. I'm reassigning it to
librust-bindgen-dev as that's where the patch goes. In reference to your
previous statement [1] on when the bug could start to affect unstable,
We expect the code to hit mainline in Linux v6.13 (the merge window
for that one will be in a few weeks); perhaps v6.14.
I'd argue that the bindgen 0.66 -> 0.70 transition [2] is large enough
(38 packages) and the first merge window is close enough that we'd
rather backport the patch before the update to v0.70. Also, since a lot
of rust code is out of tree, I think it's not uncommon for developers to
cherry-pick commits not yet in mainline, which could mean even less time
before they hit the bug. This is not counting who's already working on
the affected branches, of course.
I made sure that the patch won't break testing: in libclang 16.0.6
(testing's) clang_getFileLocation and clang_getSpellingLocation are the
exact same function [3] as correctly reported in the patch's description.
[1] https://lists.debian.org/debian-rust/2024/10/msg00044.html
[2] https://salsa.debian.org/rust-team/debcargo-conf/-/issues/89
[3]
https://github.com/llvm/llvm-project/blob/llvmorg-16.0.6/clang/tools/libclang/CXSourceLocation.cpp#L303
