Is there any upstream effort to port away from it?
Just a tiny bump on this matter, by the lead developer : "Not until it is possible to retrieve rendered images by API and then only after WebVfx has been updated to support that." (June 2020) https://forum.shotcut.org/t/is-that-any-plan-for-porting-from-qt-webkit-to-qt-webengine/5816/4
It is possible to get rid of media-libs/webvfx from depends (which requires webkit). Please see: https://github.com/reagentoo/gentoo-overlay/blob/master/media-video/shotcut/shotcut-9999.ebuild#L67-L106 But as we discussed in the past this seds doesn't worthy to be in the portage tree. And I do not mind if you remove shotcut/webvfx. These projects are generally outdated and have no prospects.
The alternative to that is a 1.5MB uncompressed patch addition - is that a more suitable solution than the 40 lines of sed? In that case, I'm thinking it would make be more logical to host the patched tarball until the upstream provides some better solution.
It's a great idea to shit the tree with a 1.5MB patch. Anyway I will not do any updates for this package. I suggest to remove shotcut,webvfx from the tree.
You can put the patch in your Github/Gitlab, or we can host it in devspace, where you can just download it via SRC_URI in ebuild. But maybe the overall situation of this package needs to be resolved first.
A 1.5 MB downstream patch is never going to happen, that's a very simple no regardless of technicalities whether it is shipped in git or as tarball.
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=97213464378a284b250c6e0c1b1b956d1e79b4fd commit 97213464378a284b250c6e0c1b1b956d1e79b4fd Author: Andreas Sturmlechner <asturm@gentoo.org> AuthorDate: 2020-10-23 16:12:55 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2020-10-23 16:12:55 +0000 profiles: Mask media-libs/webvfx, media-video/shotcut for removal Bug: https://bugs.gentoo.org/688850 Bug: https://bugs.gentoo.org/744199 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org> profiles/package.mask | 6 ++++++ 1 file changed, 6 insertions(+)
Don't you think masking the package is too hasty? October, 6: https://github.com/mltframework/shotcut/commit/a44fe75a4dc7410668935cd0d3470994f5997571 "remove qtwebkit and webvfx dependencies" October, 14: https://github.com/mltframework/shotcut/commit/488fc0035466bb0a599eb82734bed9732b4c44d8 "Merge branch 'refs/heads/drop_qtwebkit'" This is in contrast with package.mask statement "no work towards QtWebEngine port at all". If you look just qtwebkit, openshot too should be masked; and I don't see too many alternatives.
openshot was ported away in master.
Even the most recent 20.09.13 release mentions deprecation/removal of qtwebkit, so I also object to the "no work towards QtWebEngine port at all" claim.
If you are sensitive to the wording I can change that; it was made according to knowledge in this thread at the time. But considering this bug was filed 1.5 years ago, and the maintainer have since declared they would no longer work on it, the masking certainly isn't 'hasty' and also won't go away unless someone agrees to do the maintenance.
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=18d950b2796ddc101e36c7bfab432588a9c23064 commit 18d950b2796ddc101e36c7bfab432588a9c23064 Author: Andreas Sturmlechner <asturm@gentoo.org> AuthorDate: 2020-11-01 08:45:11 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2020-11-01 16:59:48 +0000 profiles: Split media-video/shotcut and media-libs/webvfx package.mask Bug: https://bugs.gentoo.org/688850 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org> profiles/package.mask | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)
Created attachment 669800 [details, diff] patch to version 20.10.31 This patch works for me (amd64) with version 20.10.31 which as per https://www.shotcut.org/blog/new-release-201031/ removed QtWebKit and WebVfx. The additional use flag for media-libs/mlt is necessary as shotcut uses /usr/bin/melt to export files.
Created attachment 672703 [details] Proposed shotcut-20-10-31.ebuild upstream upgrade to version 20.10-31 removed dependency on webvfx and qtwebkit ebuild works and shotcut works on ~amd64
(In reply to Drake Donahue from comment #14) > Created attachment 672703 [details] > Proposed shotcut-20-10-31.ebuild > > upstream upgrade to version 20.10-31 removed dependency on webvfx and > qtwebkit > ebuild works and shotcut works on ~amd64 Wow, it *only* depends on media-libs/mlt now? That's wild.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0e5ddce239de8a73d82b1887b41c3c55971d4345 commit 0e5ddce239de8a73d82b1887b41c3c55971d4345 Author: Felix Neumärker <xdch47@posteo.de> AuthorDate: 2020-11-06 08:22:50 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2020-11-21 22:51:30 +0000 media-video/shotcut: version bump to 20.10.31 Closes: https://bugs.gentoo.org/688850 Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Felix Neumärker <xdch47@posteo.de> Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org> media-video/shotcut/Manifest | 1 + media-video/shotcut/shotcut-20.10.31.ebuild | 54 +++++++++++++++++++++++++++++ 2 files changed, 55 insertions(+) Additionally, it has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=58543cb3ab79bf81b35cfa1b71cdf8f55b837a8d commit 58543cb3ab79bf81b35cfa1b71cdf8f55b837a8d Author: Andreas Sturmlechner <asturm@gentoo.org> AuthorDate: 2020-11-21 22:49:30 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2020-11-21 22:49:30 +0000 media-libs/webvfx: Remove last-rited package Bug: https://bugs.gentoo.org/688850 Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org> media-libs/webvfx/Manifest | 1 - media-libs/webvfx/metadata.xml | 18 ---------- media-libs/webvfx/webvfx-1.2.0.ebuild | 64 ----------------------------------- profiles/package.mask | 5 --- 4 files changed, 88 deletions(-)