-
Notifications
You must be signed in to change notification settings - Fork 190
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 for FFmpeg 7.0 #178
Conversation
Thanks a lot for taking care of this! As for #164, I've taken a look and made a comment, but it's not really my area of expertise, would you mind reviewing it too? I think the same comments apply to this one as zmwangx/rust-ffmpeg-sys#72 - let's wait til 7.0 landed somewhere to make sure the CI doesn't complain about anything :) |
Ok the CI is all green now :) |
Awesome! Thanks a lot for working on that 🙏 merging now |
aaand just published the crate. |
does the root readme need to be updated to mention 7.0 support? right now I see it only mentions |
This PR adds support for FFmpeg 7.0. Depends on zmwangx/rust-ffmpeg-sys#72
Also, I wasn't able to find any easy way to install 7.0 in CI yet, so the CI is not updated
FFmpeg 7.0 removed a bunch of deprecated stuff, and the
ChannelLayout
is completely changed. I tried to implement new version in a least intrusive way.Also, since it's a major version bump, could you please try to review and merge #164 for 7.0? These features (especially new_with_codec) are very much needed (judging by the amount of opened PRs, and I need them myself too) and they are simple enough that merging them should be in scope even though this crate is in maintenance mode only
Thank you