Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REGRESSION(258448@main): [GTK] Missing header with libwebrtc (USE_GSTREAMER_WEBRTC=FALSE) #8628

Conversation

manuelafm
Copy link
Contributor

@manuelafm manuelafm commented Jan 13, 2023

ee71929

REGRESSION(258448@main): [GTK] Missing header with libwebrtc (USE_GSTREAMER_WEBRTC=FALSE)
https://bugs.webkit.org/show_bug.cgi?id=250573

Reviewed by Adrian Perez de Castro.

When using libwebrtc (USE_GSTREAMER_WEBRTC=FALSE) since commit
9def6e6f0258ddd9171d32bec2fffb684dbc7b8b, there's a compiler error:

---------
In file included from /home/build/WebKit/Source/WebKit/UIProcess/WebPreferences.h:33,
                 from /home/build/WebKit/Source/WebKit/UIProcess/API/glib/WebKitSettingsPrivate.h:31,
                 from /home/build/WebKit/Tools/TestWebKitAPI/Tests/WebKitGLib/TestUIClient.cpp:23:
/home/build/WebKit/WebKitBuild/Release/WebCore/PrivateHeaders/WebCore/LibWebRTCProvider.h:37:10: fatal error: webrtc/api/peer_connection_interface.h: No such file or directory
   37 | #include <webrtc/api/peer_connection_interface.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
----------

This adds the necessary directories search for a successful
compilation.

* Tools/TestWebKitAPI/glib/CMakeLists.txt:

Canonical link: https://commits.webkit.org/258892@main

a37afe8

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios   πŸ›  mac βœ… πŸ›  wpe βœ… πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug   πŸ›  gtk   πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ§ͺ api-mac   πŸ§ͺ gtk-wk2
βœ… πŸ§ͺ api-ios   πŸ§ͺ mac-wk1   πŸ§ͺ api-gtk
βœ… πŸ›  tv   πŸ§ͺ mac-wk2
βœ… πŸ›  tv-sim   πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  watch   πŸ§ͺ mac-wk2-stress
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch-sim

@manuelafm manuelafm requested a review from a team as a code owner January 13, 2023 18:23
@mcatanzaro
Copy link
Contributor

To facilitate searching for commits, please write "[GTK]" instead of "[WebKitGTK]"

Copy link
Contributor

@aperezdc aperezdc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CMake changes LGTM, but before landing please do change the tag in the subject to [GTK] as suggested by Michael.

@manuelafm manuelafm force-pushed the eng/WebKitGTK-REGRESSION258448main-Missing-header-with-libwebrtc-USE_GSTREAMER_WEBRTCFALSE branch from e729fdb to a37afe8 Compare January 13, 2023 21:17
@manuelafm manuelafm changed the title REGRESSION(258448@main): [WebKitGTK] Missing header with libwebrtc (USE_GSTREAMER_WEBRTC=FALSE) REGRESSION(258448@main): [GTK] Missing header with libwebrtc (USE_GSTREAMER_WEBRTC=FALSE) Jan 13, 2023
@aperezdc aperezdc added the merge-queue Applied to send a pull request to merge-queue label Jan 13, 2023
…REAMER_WEBRTC=FALSE)

https://bugs.webkit.org/show_bug.cgi?id=250573

Reviewed by Adrian Perez de Castro.

When using libwebrtc (USE_GSTREAMER_WEBRTC=FALSE) since commit
9def6e6, there's a compiler error:

---------
In file included from /home/build/WebKit/Source/WebKit/UIProcess/WebPreferences.h:33,
                 from /home/build/WebKit/Source/WebKit/UIProcess/API/glib/WebKitSettingsPrivate.h:31,
                 from /home/build/WebKit/Tools/TestWebKitAPI/Tests/WebKitGLib/TestUIClient.cpp:23:
/home/build/WebKit/WebKitBuild/Release/WebCore/PrivateHeaders/WebCore/LibWebRTCProvider.h:37:10: fatal error: webrtc/api/peer_connection_interface.h: No such file or directory
   37 | #include <webrtc/api/peer_connection_interface.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
----------

This adds the necessary directories search for a successful
compilation.

* Tools/TestWebKitAPI/glib/CMakeLists.txt:

Canonical link: https://commits.webkit.org/258892@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/WebKitGTK-REGRESSION258448main-Missing-header-with-libwebrtc-USE_GSTREAMER_WEBRTCFALSE branch from a37afe8 to ee71929 Compare January 13, 2023 22:35
@webkit-commit-queue
Copy link
Collaborator

Committed 258892@main (ee71929): https://commits.webkit.org/258892@main

Reviewed commits have been landed. Closing PR #8628 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit ee71929 into WebKit:main Jan 13, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants