-
Notifications
You must be signed in to change notification settings - Fork 51
Update to FFMPEG 8.0.0 #433
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
Conversation
|
FFMPEG_jll update JuliaPackaging/Yggdrasil#12390 |
2304b0b to
19abf71
Compare
749a13c to
0431628
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #433 +/- ##
==========================================
- Coverage 78.29% 77.60% -0.69%
==========================================
Files 10 10
Lines 1313 1313
==========================================
- Hits 1028 1019 -9
- Misses 285 294 +9 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@yuyichao this is looking good to me. I'll merge and release in a day or so, unless you see anything else that needs doing? |
|
Just in case anything here strikes you as needing fixing, JET finds some issues in libffmpeg on this branch (similar on master) |
00c922d to
bf49de7
Compare
Generated using an external installation of ffmpeg.
I did need to do a manual edit since the definition for
AVVkFrameInternalwas generated after it's first use. I assume this is because the header did not forward declareAVVkFrameInternalbefore using it asstruct AVVkFrameInternal *internalinAVVkFrame. This is perfectly legal C and maybe clang.jl generator doesn't handle this too well...Passes all the tests with #432