rusty_ffmpeg icon indicating copy to clipboard operation
rusty_ffmpeg copied to clipboard

add support ffmpeg 6.1

Open peacess opened this issue 2 years ago • 5 comments

peacess avatar Nov 12 '23 23:11 peacess

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

ldm0 avatar Nov 13 '23 04:11 ldm0

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

whamer100 avatar Dec 13 '23 07:12 whamer100

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

ldm0 avatar Dec 13 '23 07:12 ldm0

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

ldm0 avatar Dec 13 '23 07:12 ldm0

@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

whamer100 avatar Dec 13 '23 07:12 whamer100