Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix build against clang 16 #5

Merged
merged 1 commit into from
Feb 10, 2023

Conversation

mqudsi
Copy link
Contributor

@mqudsi mqudsi commented Feb 10, 2023

This patch upgrades the bindgen dependency to version 0.64 which includes a fix for an upstream issue broken builds when building w/ clang 16 or newer [0].

The use of a wildcard * was removed in bindgen 0.62 [1], code has been updated to use the regex approach instead to continue working.

[0]: rust-lang/rust-bindgen#2316
[1]: rust-lang/rust-bindgen#2373

This patch upgrades the `bindgen` dependency to version 0.64 which includes a
fix for an upstream issue broken builds when building w/ clang 16 or newer [0].

The use of a wildcard `*` was removed in bindgen 0.62 [1], code has been updated
to use the regex approach instead to continue working.

[0]: rust-lang/rust-bindgen#2316
[1]: rust-lang/rust-bindgen#2373
@mqudsi
Copy link
Contributor Author

mqudsi commented Feb 10, 2023

If you can include this before you release the ffmpeg-sys-the-third crate update, that would be awesome.

@shssoichiro
Copy link
Owner

I should've read this before I published 😂 that's okay there can be a 1.0.2

@shssoichiro shssoichiro merged commit d1fea08 into shssoichiro:master Feb 10, 2023
@shssoichiro
Copy link
Owner

sys 1.0.2 is published with both fixes

@mqudsi
Copy link
Contributor Author

mqudsi commented Feb 11, 2023

Thanks ❤️

mqudsi added a commit to mqudsi/Av1an that referenced this pull request Feb 11, 2023
The recent updates to the ffmpeg dependencies broke the `ffmpeg_static` feature
to request linking against a static build of ffmpeg due to an upstream change in
how the ffmpeg repo tagged releases [0].

The v1.0.2 of the transitive ffmpeg-sys-the-third dependency includes the needed
fix to get static ffmpeg builds to work again (at all) as well as to get static
builds working on clang 16 [1].

[0]: shssoichiro/ffmpeg-the-third#3
[1]: shssoichiro/ffmpeg-the-third#5
mergify bot pushed a commit to master-of-zen/Av1an that referenced this pull request Feb 11, 2023
The recent updates to the ffmpeg dependencies broke the `ffmpeg_static` feature
to request linking against a static build of ffmpeg due to an upstream change in
how the ffmpeg repo tagged releases [0].

The v1.0.2 of the transitive ffmpeg-sys-the-third dependency includes the needed
fix to get static ffmpeg builds to work again (at all) as well as to get static
builds working on clang 16 [1].

[0]: shssoichiro/ffmpeg-the-third#3
[1]: shssoichiro/ffmpeg-the-third#5
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants