Skip to content
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

Add VideoPlayer Extension to All Platforms #2107

Closed
wants to merge 246 commits into from
Closed

Add VideoPlayer Extension to All Platforms #2107

wants to merge 246 commits into from

Conversation

time-killer-games
Copy link
Contributor

@time-killer-games time-killer-games commented Aug 10, 2020

  • Win32 games need MSYS2 installed along with the mpv/libmpv package to get all the deps.
  • Mac users need to install mpv/libmpv via homebrew or similar package managers available.
  • Linux and FreeBSD users may simply use their package manager as usual to install libmpv.
  • Android support is also possible due libmpv has an Android build - so we have all platforms.

@time-killer-games
Copy link
Contributor Author

time-killer-games commented Dec 17, 2020

@JoshDreamland halp

@time-killer-games
Copy link
Contributor Author

time-killer-games commented Dec 17, 2020

@fundies I tried both $(PKG-CONFIG) and lower case both give me big doo doo

time-killer-games added 2 commits Jan 3, 2021
@time-killer-games
Copy link
Contributor Author

time-killer-games commented Jan 3, 2021

@JoshDreamland libmpv autodetects when youtube-dl is installed in the case where the user passed a supported URL to video_add() and so i added youtube-dl as a dependency to Travis and Appveyor so when the time comes that I add tests, there can be a youtube URL video test as well. The end user does not need youtube-dl installed unless they want to load URL's, otherwise passing a URL to libmpv when youtube-dl isn't installed it will not prevent the EXE from running, it will just fail silently.

@fundies fundies closed this Jan 6, 2021
@fundies fundies reopened this Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants