Bug 62280

Summary: [Qt] Build fix for building QtWebKit inside of Qt.
Product: WebKit Reporter: Yael <yael>
Component: WebKit QtAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abecsi, ademar, kling, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch.
none
Patch. none

Description Yael 2011-06-08 07:07:57 PDT
When building QtWebKit inside of Qt, -lwebcore and -lwebkit2 are wrongly added to QtWebKit.prl, but they should not.
Comment 1 Yael 2011-06-08 07:15:01 PDT
Created attachment 96418 [details]
Patch.
Comment 2 Andras Becsi 2011-06-08 07:26:53 PDT
Comment on attachment 96418 [details]
Patch.

I would also add the CONFIG -= staticlib line to JSC.pri for consistency reasons.

The cause why the build works without it is the order of prepend calls in the QtWebKit.pro file:

!v8:prependJavaScriptCoreLib(../../JavaScriptCore)
prependWebCoreLib(../../WebCore)
webkit2:prependWebKit2Lib(../../WebKit2)

Theoretically only the last would have to remove the staticlib, but if we for some reason change the order or use these functions elsewhere then the build might fail again, so it's safer to be consistent.
Comment 3 Yael 2011-06-08 07:26:58 PDT
Created attachment 96419 [details]
Patch.

Added the same to JavaScriptCore.pri based on bbandix comment on IRC.
Comment 4 Andras Becsi 2011-06-08 07:29:23 PDT
Comment on attachment 96419 [details]
Patch.

LGTM. 
Thanks for fixing this.
Comment 5 Andreas Kling 2011-06-08 07:32:14 PDT
Comment on attachment 96419 [details]
Patch.

rs=me
Comment 6 WebKit Review Bot 2011-06-08 08:28:39 PDT
Comment on attachment 96419 [details]
Patch.

Clearing flags on attachment: 96419

Committed r88350: <http://trac.webkit.org/changeset/88350>
Comment 7 WebKit Review Bot 2011-06-08 08:28:44 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Ademar Reis 2011-06-08 08:54:15 PDT
Revision r88350 cherry-picked into qtwebkit-2.2 with commit 55ed382 <http://gitorious.org/webkit/qtwebkit/commit/55ed382>