Bug 41604 – [Qt] WebKit2 build system for Qt

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Balazs Kelemen]

Since we have started to check in sources for the Qt port of Webkit2, it is time to set up the build.

[Balazs Kelemen]

Created attachment 60513 [details]
Initial work

This comes from antti's branch on gitorious. This patch does not contain the pro file for the MiniBrowser, just the build system for WebKit2 and the WebProcess. There are two main problems with the build now:
  * Usage of precompiled header. Since the common - not platform specific - files in WebKit2 does not include everything they need, we cannot ignore to use WebKit2Prefix.h as precompiled header. The main problem I see with this is that distcc does not supports precompiled headers.
  * Now we copy every wk2 header into the build directory. We do this because the common files include the headers in the form "#include WebKit2/xyz.h" but the files are in a subdirectory of WebKit2 (for example WebKit2/UIProcess/xyz.h), so we create the WebKit2 directory in the build directory. We use this solution for the WebKit API files too, but there we just create files that includes the appropriate headers. This way incremental build works with the API headers, in contrast to the wk2 headers. A qmake wizard should do the magic for the wk2 headers as well.

This patch is not for review, since I hope we can make it better. I trust in the Trolls! :)
Actually, you cannot build WebKit2 in the trunk with this patch because there are a lot of files that are missing.

[Kenneth Rohde Christiansen]

I hope you took the build system from the rebase-after-upstream branch, because that is the most uptodate one.

[Balazs Kelemen]

Yep! :)

[Antti Koivisto]

I think webkit2 should be moved to #include "config.h" style system so it wouldn't need prefix headers. That might deserve a bug of its own.

[Antti Koivisto]

Created attachment 60890 [details]
here are build file diffs for current trunk

[Balazs Kelemen]

I think we should push this to trunk and deal with the build environment problems later. It would be much more straightforward to work on the trunk instead of pushing changes from our gitorious branches.

[Zoltan Herczeg]

(In reply to comment #6)

Totally agree. Especially since major parts have already been landed.

[Simon Hausmann]

Created attachment 61374 [details]
Updated patch for trunk

Updated patch for trunk

[Balazs Kelemen]

Simon, is it ready for review?

[Simon Hausmann]

Committed r63206: <http://trac.webkit.org/changeset/63206>