Bug 226587

Summary: Fix #include path for some Glib header files
Product: WebKit Reporter: Mohammad Imran <mohdimran.webkit>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Major CC: bugs-noreply, dvpdiner2, mcatanzaro
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Other   
OS: Linux   
Attachments:
Description Flags
Patch file to update correct path for glib header files on WebKitGTK port
mcatanzaro: review-
WebKit error build log none

Description Mohammad Imran 2021-06-03 08:31:46 PDT
Created attachment 430473 [details]
Patch file to update correct path for glib header files on WebKitGTK port

The following *.cpp files were using incorrect include path for their respective glib header files. This was causing missing header files errors while building WebkitGTK port on Ubuntu 20.04.

1. Tools/TestWebKitAPI/glib/WebKitGLib/WebViewTest.cpp - 
2. Tools/TestWebKitAPI/Tests/WebKitGLib/WebProcessTest.cpp
3. Tools/TestWebKitAPI/Tests/WebKitGLib/TestUIClient.cpp

This patch include paths to use their correct <> path.
Comment 1 Michael Catanzaro 2021-06-03 08:48:07 PDT
Comment on attachment 430473 [details]
Patch file to update correct path for glib header files on WebKitGTK port

I'm afraid you're going to have to investigate the CMake build system to see what include paths are missing and where. We don't use multi-level includes like this.
Comment 2 Darryl Pogue 2021-06-03 14:53:39 PDT
Created attachment 430501 [details]
WebKit error build log

I'm also seeing this error with the header includes. I did a fresh clone of WebKit this morning and ran the following:

  Tools/gtk/install-dependencies
  Tools/Scripts/update-webkitgtk-libs
  Tools/Scripts/build-webkit --gtk --makeargs="-j2"

Build failure error log attached.
Comment 3 Darryl Pogue 2021-10-17 17:16:53 PDT
This was fixed in https://commits.webkit.org/242819@main
Comment 4 Michael Catanzaro 2021-10-17 17:27:16 PDT
Thanks for noticing.