Bug 39006

Summary: [Qt] Link error in debug mode without debug_and_release config
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: Tools / TestsAssignee: QtWebKit Unassigned <webkit-qt-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: hausmann, jturcotte
Priority: P3 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: Windows XP   
Attachments:
Description Flags
Patch kenneth: review+, ossy: commit-queue-

Description Csaba Osztrogonác 2010-05-12 11:05:42 PDT
I experimented with removing default debug_and_release config from win32-g++/qmake.conf . 
Now qmake runs twice unnecessarly and generate makefiles for release and debug mode too.

After removing debug build broke because of a little bug. Fix is coming soon.
Comment 1 Csaba Osztrogonác 2010-05-12 11:11:00 PDT
Created attachment 55872 [details]
Patch
Comment 2 Csaba Osztrogonác 2010-05-12 11:18:25 PDT
-                CONFIG(debug, debug|release):build_pass: QTWEBKITLIBNAME = $${QTWEBKITLIBNAME}d
+                CONFIG(debug, debug|release):QTWEBKITLIBNAME = $${QTWEBKITLIBNAME}d

The QtLauncher linking break caused in debug mode caused the line above,
because if config doesn't contain debug_and_release, build_pass will
always false and QTWEBKITLIBNAME won't have necessarily "d" suffix.

Removing build_pass guard fix this bug and won't change anything.

I know modifying mkspec isn't the good solution, but it was an
experiment in first step. And I found it works. I'm going to
remove debug_and_release config from the WebKit build scripts
to decrease the runtime of building and avoid generating 
unnecessarily makefiles in an other bug.
Comment 3 Csaba Osztrogonác 2010-05-12 11:19:19 PDT
Comment on attachment 55872 [details]
Patch

cq- -ed, because I would like to land it manually.
Comment 4 Kenneth Rohde Christiansen 2010-05-12 13:42:19 PDT
Comment on attachment 55872 [details]
Patch

Be aware that this will conflict with Laszlo's patch on the cq.
Comment 5 Csaba Osztrogonác 2010-05-12 14:10:12 PDT
Landed, conflict resolved manually: http://trac.webkit.org/changeset/59266