[Webkit-unassigned] [Bug 88321] [Qt][Win] Fix broken QtWebKit5.lib linking

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=88321





--- Comment #21 from Tor Arne Vestbø <vestbo at webkit.org>  2012-06-12 05:47:19 PST ---
(In reply to comment #20)
> Isn't the basic issue that for _each_ sublibrary we do
> 
> DEFINES += BUILDING_WEBKIT
> 
> in Tools/qmake/mkspecs/features/default_post.prf, whereas BUILDING_WEBKIT should only be defined in the final linking phase of the dll?
> 
> At least I remember having similar issues with the old build system which were caused by the wrongly defined BUILDING_WEBKIT.


BUILDING_WEBKIT controls the exports for the final QtWebKit library (QWEBKIT_EXPORT), which we have spread out in multiple static libs,  and we might use between static libs. So defining BUILDING_WEBKIT for all the static libs is correct AFAICT, so that they are all dllexport.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.