Bug 55213 – [Qt] Make the WebKit2 build system less confusing for non-Qt developers

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Andras Becsi]

Move the WebKit2 API sources from WebKit/qt/QtWebKit.pro into WebKit2/WebKit2API.pri and include that in the final project file to make it easier to localize where to add new sources.

[Benjamin Poulain]

That is a good idea. Yesterday, I was lost with the WebKit2 sources being in WebKit/qt/QtWebKit.pro. I think having them in the WebKit2 dir would be far less confusing.

[Andras Becsi]

Created attachment 83798 [details]
proposed patch

[Laszlo Gombos]

What about moving these blocks to the existing WebKit2.pri instead of creating yet another build file (WebKit2API.pri) ?

[Andras Becsi]

(In reply to comment #3)
> What about moving these blocks to the existing WebKit2.pri instead of creating yet another build file (WebKit2API.pri) ?

I think in this case it is more explicit that those sources are the API, and makes it easier to find out where to add new API sources to.
But if adding a new file is a problem, I can surely move these blocks to WebKit2.pri, but I think that would make them almost as much hidden as they were in QtWebKit.pro.

[Csaba Osztrogonác]

I don't have a strong opinion, WebKit2.pri and WebKit2API.pri are good for me.

[Balazs Kelemen]

It is all about not confusing non-Qt guys and WebKit2API.pri speaks for itself
so I think it is better.

[Andras Becsi]

Created attachment 83805 [details]
proposed patch v2

[Balazs Kelemen]

I won't fight with the Apple guys even more! I would like to go with the first approach.

[Csaba Osztrogonác]

After think it over again, adding sources and headers to WebKit2.pri 
and WebKit2.pro can be confusing for non-Qt developers. But the name 
of WebKit2API.pri must be obviously for everyone.

So that's why I'd like to r+ the first one.

[Andras Becsi]

Comment on attachment 83805 [details]
proposed patch v2

Removing r?, waiting for Ossy.

[Benjamin Poulain]

I also prefer having a new file like WebKit2API.pri (maybe the name isn't perfect, WebKit2ApiSource.pri?)

WebKit2.pri contains build details that should not be messed with. This file will likely get more and more complicated over time.

I think it is cleaner to have a separate file to reference the source files.

[Csaba Osztrogonác]

Comment on attachment 83798 [details]
proposed patch

Let's go ahead, r=me.

[Csaba Osztrogonác]

Comment on attachment 83798 [details]
proposed patch

Clearing flags on attachment: 83798

Committed r79690: <http://trac.webkit.org/changeset/79690>

[Csaba Osztrogonác]

All reviewed patches have been landed.  Closing bug.