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

add support ffmpeg 6.1 #100

Closed
peacess opened this issue Nov 12, 2023 · 5 comments · Fixed by #114
Closed

add support ffmpeg 6.1 #100

peacess opened this issue Nov 12, 2023 · 5 comments · Fixed by #114
Assignees

Comments

@peacess
Copy link

peacess commented Nov 12, 2023

No description provided.

@peacess peacess changed the title add suppot ffmpeg 6.1 add support ffmpeg 6.1 Nov 12, 2023
@ldm0
Copy link
Collaborator

ldm0 commented Nov 13, 2023

I've locally tested ffmpeg 6.1 with rusty_ffmpeg yesterday, and everything seems fine.

@whamer100
Copy link

whamer100 commented Dec 13, 2023

im having issues getting 6.1 to be used, im getting this error:

error[E0587]: type has conflicting packed and align representation hints
 --> C:\Projects\Rust\rs_gtproj\target\debug\build\rusty_ffmpeg-1acc2c7ce02b39bd\out/binding.rs:3:81514
  |
3 | ... # [repr (C , packed (8))] # [repr (align (8))] # [derive (Copy , Clone)] pub union __mingw_ldbl_type_t { pub x : u128 , pub lh : __mingw_ldbl_type_t__bindgen_ty_1 , } # [repr (C... 
  |         

any suggestions? im using cargo-vcpkg on windows with the line x86_64-pc-windows-msvc = { triplet = "x64-windows-static-md", dependencies = ["ffmpeg"] } for the package.metadata.vcpkg.target bit in the toml

im relatively new to rust so feel free to point out if im straight up just doing something wrong

@ldm0 ldm0 self-assigned this Dec 13, 2023
@ldm0
Copy link
Collaborator

ldm0 commented Dec 13, 2023

@whamer100 Nope, you didn't do anything wrong. It seems to be a bindgen bug: rust-lang/rust-bindgen#2159.

@ldm0
Copy link
Collaborator

ldm0 commented Dec 13, 2023

I am going to reproduce it on my windows machine and create a patch later.

@whamer100
Copy link

@whamer100 Nope, you didn't do anything wrong. It seems to be a bindgen bug: rust-lang/rust-bindgen#2159.

good to know, and thank you

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 a pull request may close this issue.

3 participants