Navigation Menu

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

mpv fails to build with --enable-dvbin on linux 5.14+ #9233

Closed
decathorpe opened this issue Sep 22, 2021 · 3 comments · Fixed by #9234
Closed

mpv fails to build with --enable-dvbin on linux 5.14+ #9233

decathorpe opened this issue Sep 22, 2021 · 3 comments · Fixed by #9234
Labels

Comments

@decathorpe
Copy link

mpv version and platform versions

affects mpv 0.33.1, and the related source files have not been touched since then.

Reproduction steps

Building mpv with --enable-dvbin fails with linux 5.14 kernel headers, because the /usr/include/linux/dvb/video.h was (re)moved, which is included in mpv's stream/dvbin.h.

Looking at recent linux kernel headers, this file seems to have been removed without replacement. Not sure what the expected action for application developers is.

Expected behavior

mpv should continue to compile (though I'm not sure how to resolve this issue, since the kernel headers are just gone, without obvious replacement).

Actual behavior

mpv fails to compile with --enable-dvbin.

Log file

Build logs don't contain anything helpful other than the obvious "file /usr/include/linux/dvb/video.h not found".

@CounterPillow
Copy link
Contributor

Exists on my 5.14 system, and elixir seems to agree with me here: https://elixir.bootlin.com/linux/v5.14.7/source/include/uapi/linux/dvb/video.h

What seems more likely to me is that your kernel is now built without dvbin support, as staging drivers are disabled by default.

@CounterPillow
Copy link
Contributor

CounterPillow commented Sep 22, 2021

I see, there is indeed some 5.14 regression regarding DVB header files, here's the LWN article about it: https://lwn.net/Articles/867275/

Maybe it's finally time to bin dvbin.

EDIT: fwiw, mpv builds for me on 5.14.6 with --enable-dvbin. If the specific header we're having an issue with is really only used by one 15 year old piece of hardware, then it should most likely be dropped from mpv.

EDITEDIT: looks like we can just drop the include, we don't even use anything from the header; builds just fine without it.

CounterPillow added a commit to CounterPillow/mpv that referenced this issue Sep 22, 2021
We do not actually use this, and it's in a state of maybe-removal
from the kernel as of Linux 5.14. Get rid of it; mpv still builds
fine without it, so it wasn't needed anyways.

Fixes mpv-player#9233.
CounterPillow added a commit to CounterPillow/mpv that referenced this issue Sep 22, 2021
We do not actually use this, and it's in a state of maybe-removal
from the kernel as of Linux 5.14. Get rid of it; mpv still builds
fine without it, so it wasn't needed anyways.

Fixes mpv-player#9233.
CounterPillow added a commit to CounterPillow/mpv that referenced this issue Sep 22, 2021
In Linux kernel commit 819fbd3d8ef36c09576c2a0ffea503f5c46e9177
these two header files were moved to staging (though they've since
been moved out again by Linus.)

We do not actually use this, and it's in a state of maybe-removal
from the kernel as of Linux 5.14. Get rid of it; mpv still builds
fine without it, so it wasn't needed anyways.

Fixes mpv-player#9233.
@decathorpe
Copy link
Author

Well, if those includes aren't needed at all, that's like, the best-case scenario :) Thanks for investigating.

CounterPillow added a commit to CounterPillow/mpv that referenced this issue Sep 22, 2021
In Linux kernel commit 819fbd3d8ef36c09576c2a0ffea503f5c46e9177
these two header files were moved to staging (though they've since
been moved out again by Linus.)

We do not actually use this, and it's in a state of maybe-removal
from the kernel as of Linux 5.14. Get rid of it; mpv still builds
fine without it, so it wasn't needed anyways.

Fixes mpv-player#9233.
jeeb pushed a commit that referenced this issue Sep 22, 2021
In Linux kernel commit 819fbd3d8ef36c09576c2a0ffea503f5c46e9177
these two header files were moved to staging (though they've since
been moved out again by Linus.)

We do not actually use this, and it's in a state of maybe-removal
from the kernel as of Linux 5.14. Get rid of it; mpv still builds
fine without it, so it wasn't needed anyways.

Fixes #9233.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants