Summary: | media-tv/mythtv-32.0-r3 fails to build with >=media-libs/libva-2.18 (opengl/mythvaapiinterop.h:25:10: fatal error: va/va_glx.h: No such file or directory) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Marius Stoica <Letto2> |
Component: | Current packages | Assignee: | Wilson M. Michaels <thebitpit> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | asturm, darkdefende, florian-evers, leonchik1976, Letto2, mattst88, neil, proxy-maint, sam, vaapi |
Priority: | Normal | Keywords: | PullRequest |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://github.com/gentoo/gentoo/pull/31432 https://github.com/MythTV/mythtv/pull/760 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build.log
33.1 ebuild mythtv-33.1-libva.patch |
Description
Marius Stoica
2023-04-17 08:05:17 UTC
Created attachment 860225 [details]
build.log
Of course, this only happens if the vaapi use flag is enabled. Also, media-tv/mythtv[vaapi] can be emerged just fine with media-libs/libva-2.17.0 (and fails with media-libs/libva-2.18.0-r1) So, it might be a problem with the latest version of media-libs/libva ... Indeed, 2.18.0-r1 ebuild shows the config phase set to "-Dwith_x11=$(usex X) -Dwith_glx=no". Not sure why that was explicitly disabled, and the header files that were included before removed. Confirmed. I have the same issue. Downgrading to media-libs/libva-2.17.0-1 fixed it. I can confirm that setting Dwith_glx=yes in media-libs/libva-2.18.0 floowed by recompiling media-libs/libva and mythtv also fixes the issue. Tested with mythfrontend. My bad, it seems like I missed this issue. I'll try to provide a patch and try to upstream it as well. The issue seems like they unconditionally try to drag in the glx backend from libva even if the program will use the egl backend if available. (And currently on Gentoo we always have EGL available for OpenGL) I missed this when testing because I had lingering libva headers on my system. Created attachment 863763 [details]
33.1 ebuild
Created attachment 863764 [details, diff]
mythtv-33.1-libva.patch
Could you guys checks if it works for you with the provided 33.1 ebuild and the libva patch? The patch can be backported to version 32, but I figured we should probably try to bump the version either way. Hi Sebastian, at first, I really appreciate your efforts to fix and update MythTV for us, great work, thank you! I added your two files to my local portage overlay and was able to compile mythtv-33.1 without any issues (after grabbing the remaining stuff for the files folder). Looks great. However, a test (startup) needs some time as my box running the mythbackend performs an update just now, and I need to wait for it to finish. Then, I'm going to add the local overlay and update to mythtv-33.1 to have a matching backend. I'll report as soon as I have new information! Kind regards, Florian Thank you! Sebastian, when we've had a confirmation it works, could you throw up a PR with this in? (And thanks for looking at this) Hi again, So, its done. I had no problems with the update at all. Interestingly, mythfrontend from mythtv-33.1 seems to work fine together with mychbackend from mythtv-32.0-r4 (they reside on different machines, of course). I didn't expect that, normally there were database version warnings or other issues. Even after updating mythbackend on my server everything worked fine. There was no need to invoke mythtv-setup on the backend server at all. Nevertheless, I started mythtv-stup later to check for any issues, but there were no messages regarding a database migration or incompatibilities. Beware, check your backup strategy, maybe I was a little bit careless here. That could have toasted my TV setup. To summarize, works well for me, I'm happy, thank you all :-) Regards, Florian I've opened up a pull request upstream in the mythtv repo as well, in case anyone is interested: https://github.com/MythTV/mythtv/pull/760 The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c48ae7a5696fe3ddabd170e3e6e12f1ea26dc7bb commit c48ae7a5696fe3ddabd170e3e6e12f1ea26dc7bb Author: Sebastian Parborg <darkdefende@gmail.com> AuthorDate: 2023-06-14 10:06:12 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-06-14 23:58:47 +0000 media-tv/mythtv: Bump to 33.1 and fix compilation with libva Closes: https://bugs.gentoo.org/904437 Signed-off-by: Sebastian Parborg <darkdefende@gmail.com> Closes: https://github.com/gentoo/gentoo/pull/31432 Signed-off-by: Sam James <sam@gentoo.org> media-tv/mythtv/Manifest | 1 + media-tv/mythtv/files/mythtv-33.1-libva.patch | 105 +++++++ media-tv/mythtv/mythtv-33.1.ebuild | 421 ++++++++++++++++++++++++++ 3 files changed, 527 insertions(+) The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1fd8f303366c06b64df5b7b3e9f2776482a34bd7 commit 1fd8f303366c06b64df5b7b3e9f2776482a34bd7 Author: Sam James <sam@gentoo.org> AuthorDate: 2023-06-15 00:14:20 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-06-15 00:14:20 +0000 media-tv/mythtv: add bug/upstream PR info to patch Bug: https://bugs.gentoo.org/904437 Signed-off-by: Sam James <sam@gentoo.org> media-tv/mythtv/files/mythtv-33.1-libva.patch | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) |