Opened 11 months ago
Closed 11 months ago
#68963 closed defect (fixed)
mpv @0.37.0 is broken on most of the systems
Reported by: | barracuda156 | Owned by: | i0ntempest |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.8.1 |
Keywords: | bigsur, mojave, highsierra, sierra, elcapitan, yosemite, mavericks, mountainlion, lion, snowleopard, leopard | Cc: | Ionic (Mihai Moldovan) |
Port: | mpv |
Description
It is broken now pretty badly, far beyond just powerpc: https://ports.macports.org/port/mpv/details Likely just wrong settings are chosen.
If it is broken beyond repair, old systems should be pegged to the last version which builds and works.
Change History (10)
comment:1 follow-up: 4 Changed 11 months ago by kencu (Ken)
comment:2 Changed 11 months ago by i0ntempest
Seems that on every failing system except for macOS 11 it’s because of SDK version being too old. If you want to maintain support for older systems maybe it’s time to make a legacy port, as I’m unable to test and maintain stuff on old systems. But we do still have mplayer.
Also… when’s the last time we heard from Ionic?
comment:3 Changed 11 months ago by barracuda156
So portfile forces +x11 variant for old systems, which requires libplacebo, which is broken on old systems: https://trac.macports.org/ticket/68962 This makes no sense :)
comment:4 Changed 11 months ago by barracuda156
Replying to kencu:
mpv uses the latest SDK features, for the most part
all older systems should use MPlayer instead, just as good.
I cannot make it work for video output, unfortunately. This is not to claim it does not work, just a statement of fact: on my system it builds, but crashes on video playback. Audio works.
I will try building it on 10.4, if I have most of needed dependencies.
- S. AFAIR, 0.34.1 version of
mpv
actually worked.
comment:5 follow-ups: 6 7 Changed 11 months ago by i0ntempest
libplacebo is needed no matter what. +x11 is really an old remnant, if we do decide to split a legacy port out of current portfile it will be removed (from this portfile) along with other stuff that used to work on old systems.
comment:6 Changed 11 months ago by barracuda156
Replying to i0ntempest:
libplacebo is needed no matter what. +x11 is really an old remnant, if we do decide to split a legacy port out of current portfile it will be removed (from this portfile) along with other stuff that used to work on old systems.
I did not dig into the code yet, but I suspect that it will be a pointless endeavor to try fixing what was broken in recent versions, and rather makes sense to peg the last functional version as it were.
If no one is gonna bother fixing it in coming days, I can verify that what works and restore a fallback. Rather have some version working than leave it broken indefinitely.
comment:7 Changed 11 months ago by barracuda156
Replying to i0ntempest:
libplacebo is needed no matter what.
It was just added by you in 0.37.0, it seems: https://github.com/macports/macports-ports/commit/155d173926353818286c372661b3071055e88afa
comment:8 Changed 11 months ago by barracuda156
So there are two problems with 0.36.0:
- A commit which has broken patches: https://github.com/macports/macports-ports/commit/b6cb56f5d86f11127102ed6b0c1f66d8d78bbf0b
- Issue #67903
comment:9 Changed 11 months ago by barracuda156
UPD. Well, upstream deliberately broken 0.37.0 with https://github.com/mpv-player/mpv/commit/23de1deaaaace05bb9504567a852ac66e76a1ad1 No surprise it is in fact broken.
comment:10 Changed 11 months ago by barracuda156
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
mpv uses the latest SDK features, for the most part
all older systems should use MPlayer instead, just as good.