diff options
author | Dudemanguy <random342@airmail.cc> | 2022-01-24 22:53:02 -0600 |
---|---|---|
committer | Dudemanguy <random342@airmail.cc> | 2022-02-07 21:17:12 +0000 |
commit | f9bf6a601c563015706bed7bdb2b4984119db360 (patch) | |
tree | 1ca288adef7615020794db7e6cee9a8f110650c5 /video/out | |
parent | 649556b2b65207c0d40751fae941223978b04932 (diff) | |
download | mpv-f9bf6a601c563015706bed7bdb2b4984119db360.tar.bz2 mpv-f9bf6a601c563015706bed7bdb2b4984119db360.tar.xz |
meson: remove horrifying macos luajit hack
See the previous commit for the full explanation. Basically, luajit 2.0
has a bad pc file on macos that causes libmpv to fail during build. The
workaround was, if the os was darwin and luajit was found, to save a
full luajit dep and a partial luajit dep with the link args removed. The
partial dep was used for compiling libmpv, and the full dep was used for
the actual mpv executable. This worked and was needed for the CI to pass
but it sucked. Since the previous commit now makes the CI grab lua 5.1,
we don't need all this crap anymore. Just delete it and treat the
dependency normally.
This does effectively mean that building libmpv with luajit 2.0 on macOS
will no longer work with the meson build. However libraries not being
built correctly is not a mpv-specific issue. The waf build will succeed
for some reason, but it has known issues and it would be better if it
just failed honestly. An upstream developer said years ago that that
macOS users should use the 2.1 branch (and there's no release of
course). In any case, no macOS user should be building mpv with luajit
2.0, so we shouldn't be going out of our way to support this.
https://github.com/mpv-player/mpv/issues/7512
https://github.com/LuaJIT/LuaJIT/issues/521#issuecomment-562999247
Diffstat (limited to 'video/out')
0 files changed, 0 insertions, 0 deletions