[Pkg-rust-maintainers] Bug#1086510: Bug#1086510: bindgen: bug triggers future kernel build error (fixed upstream)

[NoisyCoil]

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