-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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
[ffmpeg] add usage #17284
[ffmpeg] add usage #17284
Conversation
e09a545
to
c5c304f
Compare
Probably no need for |
Yeah that is right, but I don't know if that is allowed |
@autoantwort ,please run the vcpkg x-add-version ffmpeg --overwrite-version command then commit angin. |
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.
@autoantwort ,please set Port-Version: 3 to Port-Version: 4
@autoantwort I think you should go with this first, since it already LGTM. |
Ok, done |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Note to self: merge at same time as #17299 |
6358f3f
to
240d055
Compare
# Conflicts: # versions/f-/ffmpeg.json
@autoantwort Can you resolve the conflicts? |
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.
Thanks for your PR!
Describe the pull request
What does your PR fix?
How to use ffmpeg (see discussion on discord)
Which triplets are supported/not supported? Have you updated the CI baseline?
All
Does your PR follow the maintainer guide?
Yes
If you have added/updated a port: Have you run
./vcpkg x-add-version --all
and committed the result?I am still working on this PR
After #17153, #17204 and #17236 are merged, I will run
./vcpkg x-add-version --all
.