-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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] enable opengl feature on osx #18624
[ffmpeg] enable opengl feature on osx #18624
Conversation
Hi @mcmtroffaes Thanks again for your PR.
Seems this is still needed to update. vcpkg/ports/ffmpeg/portfile.cmake Lines 75 to 76 in 6e024e7
|
Oh yes, I missed it as I removed these checks in my CI to allow testing the more experimental features. Fixed now. |
Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
You should have tested this feature |
Well, I don't have a mac myself for in-depth testing, however I've run a simple test application on CI https://github.com/mcmtroffaes/ffmpeg-msvc-build/runs/2903919175#step:14:1122 which shows that the opengl device can be opened on osx. |
The feature
Thanks again for your efforts @mcmtroffaes. |
You're welcome, and thanks for running an extra local test. I'm pretty confident that if it works on CI it should also work for users but better be safe than sorry! |
Merged master & bumped port version again. |
# Conflicts: # ports/ffmpeg/vcpkg.json # versions/baseline.json # versions/f-/ffmpeg.json
Thanks for your contribution! |
What does your PR fix?
The opengl feature was disabled on osx when compiling ffmpeg[all], however ffmpeg supports it perfectly on osx and it compiles with no problems (see for instance https://github.com/mcmtroffaes/ffmpeg-msvc-build/runs/2903919175).
Which triplets are supported/not supported? Have you updated the CI baseline?
No changes besides adding an extra feature for osx.
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?Yes.