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

Missing OSC: How to get lua development headers #5200

Closed
no-simpler opened this issue Dec 8, 2017 · 6 comments
Closed

Missing OSC: How to get lua development headers #5200

no-simpler opened this issue Dec 8, 2017 · 6 comments

Comments

@no-simpler
Copy link

no-simpler commented Dec 8, 2017

mpv version and platform

mpv 0f9a690 installed on macos 10.13.1 via Homebrew with:

brew install mpv --HEAD --with-bundle --with-libaacs --with-libarchive \
       --with-libcaca --with-libdvdnav --with-pulseaudio --with-rubberband \
       --with-uchardet --with-vapoursynth

Reproduction steps

mpv -v --osc file.mp4

Expected behavior

Plays video with OSC.

Actual behavior

[cplayer] Error parsing option osc (option not found)
[cplayer] Setting commandline option --osc=yes failed.
[cplayer]
[cplayer] Exiting... (Fatal error)

Log file

mpv -v --osc file.mp4: http://sprunge.us/KVAE

mpv -v: http://sprunge.us/eYMC

Note

I’ve read the FAQ:

Install Lua (including development headers) and rebuild mpv.

Lua 5.3.4 is installed of course. How do I get Lua development headers?

Sorry to ask, I know I’m in over my head with this. Thanks in advance if you spare a moment to help me out.

@selsta
Copy link
Contributor

selsta commented Dec 8, 2017

Homebrew/homebrew-core#21389

@Argon-
Copy link
Member

Argon- commented Dec 8, 2017

mpv does not support 5.3. You need 5.1 or 5.2 (both work, I've used both long enough).
To get a working version of mpv with brew, all you need to do is change one single line in the brew formula. The brew maintainers know this too but they don't want to fix it because the person who asked apparently didn't ask nice enough (see the link above).
Please complain to them.

@Argon-
Copy link
Member

Argon- commented Dec 8, 2017

To fix it locally: brew edit mpv
Search the line depends_on "lua" => :recommended and replace it with depends_on "lua@5.1" then rebuild from source (in your case, just install as you did before).

@CounterPillow
Copy link
Contributor

I've adjusted the FAQ page to explicitly mention the supported Lua versions.

@no-simpler
Copy link
Author

Dear lord, this package was hard to install.

It worked, of course; replies are much appreciated.

@ilovezfs
Copy link

ilovezfs commented Dec 9, 2017

See my comments here: #5080 (comment)

TL;DR I've merged the 5.1 PR ☮️ 🕊️ ✌️

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

No branches or pull requests

5 participants