WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 209209
[CMake] libopus 1.1 its enough for building WebKitGTK with ENABLE_WEB_RTC
https://bugs.webkit.org/show_bug.cgi?id=209209
Summary
[CMake] libopus 1.1 its enough for building WebKitGTK with ENABLE_WEB_RTC
Carlos Alberto Lopez Perez
Reported
2020-03-17 19:01:37 PDT
Building WebKitGTK with -DENABLE_WEB_RTC=ON on Ubuntu 18.04 its not longer possible since
r258412
This failure happens: [...] -- Using platform-specific CMakeLists: /opt/webkitgtk/nightly/build/Source/JavaScriptCore/shell/PlatformGTK.cmake -- Using source list file: Sources.txt -- Using source list file: SourcesGTK.txt -- Using platform-specific CMakeLists: /opt/webkitgtk/nightly/build/Source/ThirdParty/ANGLE/PlatformGTK.cmake -- Performing Test CXX_COMPILER_SUPPORTS_-Wno-suggest-attribute=format -- Performing Test CXX_COMPILER_SUPPORTS_-Wno-suggest-attribute=format - Success -- Platform-specific CMakeLists not found: /opt/webkitgtk/nightly/build/Source/ThirdParty/gtest/PlatformGTK.cmake -- Could NOT find LibOpus: Found unsuitable version "1.1.2", but required is at least "1.3" (found /usr/lib/x86_64-linux-gnu/libopus.so) CMake Error at Source/ThirdParty/libwebrtc/CMakeLists.txt:21 (message): libopus is needed for USE_LIBWEBRTC. libopus 1.1.2 (what ubuntu 18.04 ships) its enough for -DENABLE_WEB_RTC and it was working before
r237174
because the check was not working as expected.
Attachments
Patch
(1.37 KB, patch)
2020-03-17 19:19 PDT
,
Carlos Alberto Lopez Perez
no flags
Details
Formatted Diff
Diff
Patch
(1.37 KB, patch)
2020-03-17 19:23 PDT
,
Carlos Alberto Lopez Perez
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Carlos Alberto Lopez Perez
Comment 1
2020-03-17 19:19:48 PDT
Created
attachment 393811
[details]
Patch
Carlos Alberto Lopez Perez
Comment 2
2020-03-17 19:23:49 PDT
Created
attachment 393812
[details]
Patch
Konstantin Tokarev
Comment 3
2020-03-17 19:54:46 PDT
Did you actually test it?
Carlos Alberto Lopez Perez
Comment 4
2020-03-17 20:25:08 PDT
(In reply to Konstantin Tokarev from
comment #3
)
> Did you actually test it?
I tested that it builds and that WebRTC WPT related tests pass. Check:
https://wpt.fyi/results/webrtc?label=master&label=experimental&product=chrome&product=firefox&product=safari&product=webkitgtk&aligned
The version of WebKitGTK that its tested on the WPT CI its built for Ubuntu 18.04 in this bot
https://build-webkit.igalia.com/builders/GTK%20Linux%2064-bit%20WPT%20Packaging%20%28nightly%20channel%29
And it started to fail building after
r258412
Once that bot finish building correctly it uploads the built product to
https://webkitgtk.org/built-products/
for later consuming on the WPT CI (taskcluster) that runs currently on Ubuntu 18.04 and runs all the WPT tests. Beyond that, I did not tested it further. If you know a specific test to try, let me know.
Konstantin Tokarev
Comment 5
2020-03-17 20:28:11 PDT
Comment on
attachment 393812
[details]
Patch I've actually meant something like running WebRTC on some real site, but I guess passing WPT should be enough.
WebKit Commit Bot
Comment 6
2020-03-17 21:46:01 PDT
Comment on
attachment 393812
[details]
Patch Clearing flags on attachment: 393812 Committed
r258624
: <
https://trac.webkit.org/changeset/258624
>
WebKit Commit Bot
Comment 7
2020-03-17 21:46:03 PDT
All reviewed patches have been landed. Closing bug.
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