WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
241128
[WPE][GTK] Expose ENABLE_VIDEO build option
https://bugs.webkit.org/show_bug.cgi?id=241128
Summary
[WPE][GTK] Expose ENABLE_VIDEO build option
Michael Catanzaro
Reported
2022-05-31 06:29:30 PDT
We should expose ENABLE_VIDEO as a public build option because it allows disabling GStreamer, a major dependency. This is useful on embedded systems where video is not needed.
Attachments
Add attachment
proposed patch, testcase, etc.
Michael Catanzaro
Comment 1
2022-05-31 06:31:59 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/1181
EWS
Comment 2
2022-05-31 10:06:29 PDT
Committed
r295057
(
251152@main
): <
https://commits.webkit.org/251152@main
> Reviewed commits have been landed. Closing PR #1181 and removing active labels.
Adrian Perez
Comment 3
2022-05-31 12:20:09 PDT
(In reply to Michael Catanzaro from
comment #0
)
> We should expose ENABLE_VIDEO as a public build option because it allows > disabling GStreamer, a major dependency. This is useful on embedded systems > where video is not needed.
For future reference, in case someone arrives to this bug trying to avoid the GStreamer dependency in their builds: ENABLE_VIDEO and ENABLE_WEB_AUDIO need to be *both disabled* in order for the WebKit WPE and GTK ports to avoid the GStreamer dependency.
Michael Catanzaro
Comment 4
2022-05-31 13:21:18 PDT
Oh... and that option is not public either. I suppose I'd better expose that too?
Michael Catanzaro
Comment 5
2022-05-31 17:18:20 PDT
Maybe I should change this to USE_GSTREAMER instead? That can toggle both ENABLE_VIDEO and ENABLE_WEB_AUDIO.
Michael Catanzaro
Comment 6
2022-06-02 11:11:33 PDT
ENABLE_VIDEO and ENABLE_WEB_AUDIO were both already public, in GStreamerDefinitions.cmake: WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_VIDEO PUBLIC ON) WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_WEB_AUDIO PUBLIC ON) Adrian and I were both wrong. I'll just revert this.
Michael Catanzaro
Comment 7
2022-06-02 11:13:51 PDT
Re-opening for pull request
https://github.com/WebKit/WebKit/pull/1249
EWS
Comment 8
2022-06-02 12:19:48 PDT
Committed
r295136
(
251227@main
): <
https://commits.webkit.org/251227@main
> Reviewed commits have been landed. Closing PR #1249 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug