- Notifications
You must be signed in to change notification settings - Fork900
Pipelines update for FFmpeg 5 and security fixes#756
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
67642cb tob5b1da3Compare2ff1528 tod780d93Compare| perl -wpe 'BEGIN {our ($found_blank, $bottle_block);} | ||
| if (/(^class )(FfmpegAT4)(\s.*)/) {$_=$1."FfmpegOpencv".$3."\n"; next;} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
You can leaveFfmpegOpencv here, because the previous logic was something like$2.Opencv what meantFfmpegOpencv, but wasn’t obvious. Or move back$2.
link to the previous logic (before my changes), notice$2:602a8be#diff-7d5a84b3617f8aff01b2fc673bb198946e3632d90592fd61d3e3e24ddd6dbf70L74
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Yes, it works, tried in my fork.
Log:https://github.com/opencv/opencv-python/actions/runs/3694906850/jobs/6256653970
Requiresopencv/opencv#22965 for the latest numpy |
No description provided.