Skip to content

fix: update bindgen for LLVM 22 compatibility - #808

Merged
runcom merged 12 commits into
fdo-rs:mainfrom
knecasov:update-deps-bindgen-llvm22-compat
Jul 30, 2026
Merged

fix: update bindgen for LLVM 22 compatibility#808
runcom merged 12 commits into
fdo-rs:mainfrom
knecasov:update-deps-bindgen-llvm22-compat

Conversation

@knecasov

Copy link
Copy Markdown
Contributor

What

Update dependencies to ensure compatibility with LLVM 22 and fix FTBFS caused by bindgen < 0.72.1.
Incorporates changes from #807.

Why

LLVM has been rebased to version 22. Clang 22 introduces a breaking change affecting bindgen.

Assisted-by: Claude (claude-opus-4-6)

@knecasov
knecasov force-pushed the update-deps-bindgen-llvm22-compat branch 3 times, most recently from 2063425 to 4e08fe1 Compare June 29, 2026 10:02
@knecasov
knecasov marked this pull request as draft June 29, 2026 14:42
@nullr0ute

Copy link
Copy Markdown
Contributor

This needs to be split into multiple commits.

There's pieces that change library revs, changes around IP handling, ssh, and even dropping of a bunch of code around pgSQL. All of those should be individual commits explaining what each one does.

@knecasov
knecasov force-pushed the update-deps-bindgen-llvm22-compat branch from 4e08fe1 to 3da78fe Compare July 1, 2026 08:13
@knecasov

knecasov commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

This needs to be split into multiple commits.

There's pieces that change library revs, changes around IP handling, ssh, and even dropping of a bunch of code around pgSQL. All of those should be individual commits explaining what each one does.

Thank you very much for the recommendation. I split the commit into individual commits - I cherry-picked commits from PR #807 and added one more. Please take a look if it is correctly separated now.

@knecasov
knecasov marked this pull request as ready for review July 8, 2026 07:36
runcom and others added 8 commits July 29, 2026 13:50
Fixes compiling errors as per parallaxsecond/rust-tss-esapi#634

Signed-off-by: Antonio Murdaca <antoniomurdaca@gmail.com>
Signed-off-by: Antonio Murdaca <antoniomurdaca@gmail.com>
Signed-off-by: Antonio Murdaca <antoniomurdaca@gmail.com>
Signed-off-by: Antonio Murdaca <antoniomurdaca@gmail.com>
Signed-off-by: Antonio Murdaca <antoniomurdaca@gmail.com>
Bump libcryptsetup-rs to 0.16.1 and devicemapper to 0.34.7.
Update CryptParamsReencrypt to wrap luks2 field in Some() per new API.

Signed-off-by: Klara Necasova <knecasov@redhat.com>
Assisted-by: Claude (claude-opus-4-6)
fdo-client-linuxapp requires libpq.so.5 which is not present
in the runtime container.

Signed-off-by: Klara Necasova <knecasov@redhat.com>
Assisted-by: Claude (claude-opus-4-6)
@mmartinv
mmartinv force-pushed the update-deps-bindgen-llvm22-compat branch from 240dddb to 58e1118 Compare July 29, 2026 11:50
mmartinv
mmartinv previously approved these changes Jul 29, 2026
mmartinv added 2 commits July 29, 2026 15:21
Add epel-9 and epel-10 targets to copr_build and test jobs to work
around packit/packit#2700. Simplify the
test targets from dict format with explicit distros to a plain list.

Signed-off-by: Miguel Martín <mmartinv@redhat.com>
Replace `for (_, cert)` with `for cert in self.certs.values()` to
follow idiomatic Rust and satisfy clippy.

Signed-off-by: Miguel Martín <mmartinv@redhat.com>
knecasov and others added 2 commits July 29, 2026 16:37
- remove unnecessary & before key.to_string() in format! macros
  in pg.rs and sqlite.rs

Signed-off-by: Klara Necasova <knecasov@redhat.com>
Remove unnecessary `&` references in format! macro arguments across
multiple crates, and simplify a match expression in
device_credential_locations.rs to use the `?` operator.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Miguel Martín <mmartinv@redhat.com>
@runcom
runcom merged commit d73f9bb into fdo-rs:main Jul 30, 2026
20 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants