Bug 269147 - [GTK] [WPE] git main build fails with -DENABLE_VIDEO=OFF
Summary: [GTK] [WPE] git main build fails with -DENABLE_VIDEO=OFF
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-10 13:38 PST by Mart Raudsepp
Modified: 2024-04-14 08:34 PDT (History)
3 users (show)

See Also:


Attachments
Log of failure with disabled gstreamer build options (119.43 KB, text/x-log)
2024-02-10 13:38 PST, Mart Raudsepp
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mart Raudsepp 2024-02-10 13:38:08 PST
Created attachment 469808 [details]
Log of failure with disabled gstreamer build options

Trying further after bug 269144 and bug 269145 to disable the whole gstreamer dep and see how WebKitGTK is doing instead of WPE, looks like ENABLE_VIDEO or similar is failing as well (same kind of error happens with WPE build as well).
This is a build combination that would be nice to have working, as I maintain that option in Gentoo, but if not, we can force gstreamer as well for now. Though eventually it got fixed and working in 2.42 series while being broken earlier, so I guess someone cares about it, so here's a report on it failing again in git main
Comment 1 Mart Raudsepp 2024-04-14 06:29:58 PDT
I believe this might just be missing a
WEBKIT_OPTION_DEPEND(ENABLE_VIDEO ENABLE_MEDIA_STREAM)
in OptionsGTK.cmake or higher up
Comment 2 Mart Raudsepp 2024-04-14 06:38:41 PDT
And
WEBKIT_OPTION_DEPEND(ENABLE_VIDEO ENABLE_WEB_CODECS)
then too.
Comment 3 Mart Raudsepp 2024-04-14 06:49:24 PDT
Sorry, the ENABLE_MEDIA_STREAM one was already there (apparently it's written the other way around as `WEBKIT_OPTION_DEPEND(ENABLE_MEDIA_STREAM ENABLE_VIDEO)` in WebKitFeatures.cmake), so this was a regression from 

commit 1aa24c14332324946e8aa737eb45aaba51485cd8
Author: Pablo Saavedra <psaavedra@igalia.com>
Date:   Wed Mar 1 14:51:55 2023 -0800

    Complement Enable HTMLCanvasElement::toVideoFrame with Web Codecs
    https://bugs.webkit.org/show_bug.cgi?id=252759


where ENABLE(WEB_CODECS) made VideoFrame referenced as well, but nothing ensuring VideoFrame is actually there.
Comment 4 Michael Catanzaro 2024-04-14 08:34:57 PDT
ENABLE_VIDEO is not supposed to depend on ENABLE_WEB_CODECS.

ENABLE_WEB_CODECS is probably supposed to depend on ENABLE_VIDEO.