- 
                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
base: master
Are you sure you want to change the base?
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