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
Patch (1.37 KB, patch)
2020-03-17 19:23 PDT, Carlos Alberto Lopez Perez
no flags
Carlos Alberto Lopez Perez
Comment 1 2020-03-17 19:19:48 PDT
Carlos Alberto Lopez Perez
Comment 2 2020-03-17 19:23:49 PDT
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.