Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase bindgen dependency lower bound #63

Merged
merged 1 commit into from
Apr 18, 2023
Merged

Increase bindgen dependency lower bound #63

merged 1 commit into from
Apr 18, 2023

Conversation

mulkieran
Copy link
Contributor

Increase to 0.63. This value was chosen because it is known to succeed in Fedora rawhide and Fedora 38.

0.59.0 bindgen now fails with the following error on these platforms:
process didn't exit successfully: /__w/libcryptsetup-rs/libcryptsetup-rs/target/debug/build/loopdev-cd83fd54aaebcba7/build-script-build (exit status: 101)
--- stderr
thread 'main' panicked at '"enum_(unnamed_at_/usr/include/linux/loop_h_16_1)" is not a valid Ident', /github/home/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-1.0.56/src/fallback.rs:811:9

Increase to 0.63. This value was chosen because it is known to succeed
in Fedora rawhide and Fedora 38.

0.59.0 bindgen now fails with the following error on these platforms:
 process didn't exit successfully: `/__w/libcryptsetup-rs/libcryptsetup-rs/target/debug/build/loopdev-cd83fd54aaebcba7/build-script-build` (exit status: 101)
  --- stderr
  thread 'main' panicked at '"enum_(unnamed_at_/usr/include/linux/loop_h_16_1)" is not a valid Ident', /github/home/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-1.0.56/src/fallback.rs:811:9

Signed-off-by: mulhern <amulhern@redhat.com>
@mulkieran
Copy link
Contributor Author

It looks like an increase to 0.62.0 would be enough to fix the FTBFS on f38: stratis-storage/project#589 (comment) .

@mdaffin Would you consider accepting this upgrade to bindgen dependency and making a new release of loopdev?

If 0.63.0 is too high for some purpose, then 0.62.0 would still work.

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.

2 participants