Summary: | media-video/vlc-3.0.17.3 fail to build with media-libs/dav1d-1.0.0 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Andrea Postiglione <andrea.postiglione> |
Component: | Current packages | Assignee: | Gentoo Media-video project <media-video> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | 1i5t5.duncan, Adrian.Bassett, atoth, b.buschinski, christian, deim, georgediam, jorge.ramos, kajanos, Letto2, lo48576, luke, maracay, mike, mmokrejs, oli.huber, peter.volkov, redblade7, sam, sgs, steffen.weber |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: | https://code.videolan.org/videolan/vlc/-/merge_requests/1618 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 835807 | ||
Attachments: |
build.log
patch against 3.0.17.3 |
Description
Andrea Postiglione
2022-03-22 08:41:00 UTC
(In reply to Christian Bricart from comment #1) > https://github.com/videolan/vlc/commit/ > dbf45cea2a8abdfbef897b8a71f3eb782bb1b712 * Applying dav1d-1.0.patch ... patching file modules/codec/dav1d.c Hunk #1 FAILED at 64. Hunk #2 FAILED at 401. Hunk #3 FAILED at 451. 3 out of 3 hunks FAILED -- saving rejects to file modules/codec/dav1d.c.rej [ !! ] * ERROR: media-video/vlc-3.0.17.3::local failed (prepare phase): * patch -p1 failed with /var/tmp/portage/media-video/vlc-3.0.17.3/files/dav1d-1.0.patch * * Call stack: * ebuild.sh, line 127: Called src_prepare * environment, line 2899: Called xdg_src_prepare * environment, line 3983: Called default * phase-functions.sh, line 858: Called default_src_prepare * phase-functions.sh, line 923: Called __eapi6_src_prepare * environment, line 588: Called eapply '/var/tmp/portage/media-video/vlc-3.0.17.3/files/vlc-2.1.0-fix-libtremor-libs.patch' '/var/tmp/portage/media-video/vlc-3.0.17.3/files/vlc-2.2.8-freerdp-2.patch' '/var/tmp/portage/media-video/vlc-3.0.17.3/files/vlc-3.0.6-fdk-aac-2.0.0.patch' '/var/tmp/portage/media-video/vlc-3.0.17.3/files/vlc-3.0.11.1-configure_lua_version.patch' '/var/tmp/portage/media-video/vlc-3.0.17.3/files/vlc-3.0.14-fix-live-address-api.patch' '/var/tmp/portage/media-video/vlc-3.0.17.3/files/dav1d-1.0.patch' * environment, line 1564: Called _eapply_patch '/var/tmp/portage/media-video/vlc-3.0.17.3/files/dav1d-1.0.patch' * environment, line 1502: Called __helpers_die 'patch -p1 failed with /var/tmp/portage/media-video/vlc-3.0.17.3/files/dav1d-1.0.patch' * isolated-functions.sh, line 112: Called die * The specific snippet of code: * die "$@" * * If you need support, post the output of `emerge --info '=media-video/vlc-3.0.17.3::local'`, * the complete build log and the output of `emerge -pqv '=media-video/vlc-3.0.17.3::local'`. * The complete build log is located at '/var/tmp/portage/media-video/vlc-3.0.17.3/temp/build.log'. * The ebuild environment file is located at '/var/tmp/portage/media-video/vlc-3.0.17.3/temp/environment'. * Working directory: '/var/tmp/portage/media-video/vlc-3.0.17.3/work/vlc-3.0.17.3' * S: '/var/tmp/portage/media-video/vlc-3.0.17.3/work/vlc-3.0.17.3' patching failed Created attachment 767587 [details, diff]
patch against 3.0.17.3
there have been some upstream code rework since 3.0.17.3 release.
this backported patch applies against =media-video/vlc-3.0.17.3
(In reply to Christian Bricart from comment #3) > Created attachment 767587 [details, diff] [details, diff] > patch against 3.0.17.3 > > there have been some upstream code rework since 3.0.17.3 release. > this backported patch applies against =media-video/vlc-3.0.17.3 now with your patch build fine. thanks (In reply to Christian Bricart from comment #3) > Created attachment 767587 [details, diff] [details, diff] > patch against 3.0.17.3 > > there have been some upstream code rework since 3.0.17.3 release. > this backported patch applies against =media-video/vlc-3.0.17.3 Could you submit your backport upstream please? (In reply to Sam James from comment #5) > (In reply to Christian Bricart from comment #3) > > Created attachment 767587 [details, diff] [details, diff] [details, diff] > > patch against 3.0.17.3 > > > > there have been some upstream code rework since 3.0.17.3 release. > > this backported patch applies against =media-video/vlc-3.0.17.3 > > Could you submit your backport upstream please? hm, I see https://code.videolan.org/videolan/vlc/-/merge_requests/1618. *** Bug 836330 has been marked as a duplicate of this bug. *** Hello: I'd like to add that the patch corrected the compile error on my machine with media-video/vlc-3.0.16-r8 So why isn't the ebuild changed to require an older version until the patch comes in? My previous version was media-libs/dav1d-0.4.0. After upgrade to media-libs/dav1d-1.0.0 I cannot compile vlc like others. Thank you. (In reply to Martin Mokrejš from comment #9) > So why isn't the ebuild changed to require an older version until the patch > comes in? dav1d-1.0.0 isn't stale^H^Hble yet, and current stable 0.9.2 works fine. vlc will almost certainly be either -rX bumped with a patch or version-bumped if upstream releases a version with the fix before dav1d-1.x is stabilized. Meanwhile, those running ~arch are expected to be able to do at least reasonably minimal package management occasionally, when things like this come up. Easy enough to package.mask the new version temporarily[1], as I decided to do here for this case, or plop the patch already posted here in /etc/portage/patches/media-video/vlc/ and rebuild vlc to apply it, if you prefer and as I tend to do more frequently, when a bug already has a patch available. --- [1] The temporary package.mask can be removed either after this bug gets fixed, which you'll know about if you're CCed, or you can use = to specify exactly the 1.0.0 version instead of something like >=1.0.0, and automatically have the next version or -rX bump unmasked to try again. Note that I've advisedly not added the patch given it's under a lot of discussion upstream & not merged yet. (In reply to Sam James from comment #11) > Note that I've advisedly not added the patch given it's under a lot of > discussion upstream & not merged yet. The MR might be outdated, see this commit: https://code.videolan.org/videolan/vlc/-/commit/dbf45cea2a8abdfbef897b8a71f3eb782bb1b712 I believe the patch attached here is based on this commit, see https://bugs.gentoo.org/835787#c3 Thank you for your kind and detailed answer. Still, the likely scenario is that I hit a bug, start searching around if somebody already hit the bug, then yes, maybe mask the package or downgrade another one, but that is exactly what you can anticipate already because evidently media-video/vlc-3.0.17.3-r1 and media-libs/dav1d-1.0.0 does not compile together. I still think you could save some 20 people the efforts and better prevent the vlc update or the dav1d update. That is already a chance for you to decide which way is better. Should everybody study if for example dav1d should be updated or vlc downgraded, for example, due to some security issue? You can solve these all questions for us. I am fine to report a new bug, contribute testing as much as I am able to do but this is just wasting our efforts. I will not even study if I should better downgrade the dav1d thing or lookup emerge.log to see if I just recompiled vlc or whether I hit this it was to be upgraded. So I will just neglect that, let revdep-rebuild run unfinished, orphaned media-libs/dav1d-0.4.0 files and vlc binary using them despite the updated dav1d. I just do not seen a reason why media-video/vlc-3.0.17.3-r1.ebuild is not already changed to require =<media-libs/dav1d-0.9.2 , likewise other affected vlc ebuilds. (In reply to Olivier Huber from comment #12) > (In reply to Sam James from comment #11) > > Note that I've advisedly not added the patch given it's under a lot of > > discussion upstream & not merged yet. > > The MR might be outdated, see this commit: > > https://code.videolan.org/videolan/vlc/-/commit/ > dbf45cea2a8abdfbef897b8a71f3eb782bb1b712 > > I believe the patch attached here is based on this commit, see > https://bugs.gentoo.org/835787#c3 I don't see a fix on the 3.0.x branch: https://code.videolan.org/videolan/vlc/-/commits/3.0.x. There was quite a bit of discussion on the MR about whether there's actually a bug here that they noticed while fixing the dav1d issue. That said, nobody here has complained about runtime issues, so let's add it in a revbump in ~arch. (In reply to Martin Mokrejš from comment #13) > Thank you for your kind and detailed answer. Still, the likely scenario is > that I hit a bug, start searching around if somebody already hit the bug, > then yes, maybe mask the package or downgrade another one, but that is > exactly what you can anticipate already because evidently > media-video/vlc-3.0.17.3-r1 and media-libs/dav1d-1.0.0 does not compile > together. I don't get your point. > > I still think you could save some 20 people the efforts and better prevent > the vlc update or the dav1d update. That is already a chance for you to > decide which way is better. Should everybody study if for example dav1d > should be updated or vlc downgraded, for example, due to some security > issue? Obviously security issues get more priority over inconveniencing people with a downgrade-rebuild of Firefox so I'd be happy to add a < dep in that case. > You can solve these all questions for us. I am fine to report a new > bug, contribute testing as much as I am able to do but this is just wasting > our efforts. > I don't get your point. > > I just do not seen a reason why media-video/vlc-3.0.17.3-r1.ebuild is not > already changed to require =<media-libs/dav1d-0.9.2 , likewise other > affected vlc ebuilds. The answer to this being that inflicting a downgrade on everybody is unfortunate and I was hoping the upstream MR, which had some momentum, would get merged. It did not. The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=57322daae014d657b2b063abff4af216f1d90151 commit 57322daae014d657b2b063abff4af216f1d90151 Author: Sam James <sam@gentoo.org> AuthorDate: 2022-04-07 22:32:43 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2022-04-07 22:33:17 +0000 media-video/vlc: add dav1d-1.0.0 patch It's unmerged (pending upstream) but nobody's complained of runtime issues and the patch itself wasn't disputed; just whether there's another separate runtime bug which people noticed during review of the patch. Closes: https://bugs.gentoo.org/835787 Signed-off-by: Sam James <sam@gentoo.org> .../vlc/files/vlc-3.0.17.3-dav1d-1.0.0.patch | 78 ++++ media-video/vlc/vlc-3.0.17.3-r2.ebuild | 515 +++++++++++++++++++++ 2 files changed, 593 insertions(+) (In reply to Sam James from comment #15) > (In reply to Martin Mokrejš from comment #13) > I don't get your point. > > > Obviously security issues get more priority over inconveniencing people with > a downgrade-rebuild of Firefox so I'd be happy to add a < dep in that case. > > I don't get your point. > > The answer to this being that inflicting a downgrade on everybody is > unfortunate and I was hoping the upstream MR, which had some momentum, would > get merged. It did not. Sorry for my bad English, in brief, I did not know whether I should prefer the vlc update or the dav1d downgrade, how to weigh the scenarios. Anyway, I understand now what delayed you and thank your the fix patch. |