Title: [148789] trunk
Revision
148789
Author
[email protected]
Date
2013-04-20 02:53:06 -0700 (Sat, 20 Apr 2013)

Log Message

[Qt][Mac] Remove obsolete workaround for debug builds
https://bugs.webkit.org/show_bug.cgi?id=114750

Reviewed by Jocelyn Turcotte.

This workaround made default builds fail with recent Qt5 because
it removed the major version number from the library name, producing
QtWebKitWidgets, whereas the linking command line tried to link
against Qt5WebKitWidgets.
Debug builds are possible with and without framework-enabled builds
of Qt, but the debug versions of the Qt libraries have to be present.
Debug builds with a release version of Qt are not possible on Mac
since for debug builds qmake produces a linker command line where
all the Qt libraries have the "_debug" suffix, therefore if the debug
libraries are missing the build fails.

* Source/widgetsapi.pri:

Modified Paths

Diff

Modified: trunk/ChangeLog (148788 => 148789)


--- trunk/ChangeLog	2013-04-20 08:23:19 UTC (rev 148788)
+++ trunk/ChangeLog	2013-04-20 09:53:06 UTC (rev 148789)
@@ -1,3 +1,23 @@
+2013-04-20  Andras Becsi  <[email protected]>
+
+        [Qt][Mac] Remove obsolete workaround for debug builds
+        https://bugs.webkit.org/show_bug.cgi?id=114750
+
+        Reviewed by Jocelyn Turcotte.
+
+        This workaround made default builds fail with recent Qt5 because
+        it removed the major version number from the library name, producing
+        QtWebKitWidgets, whereas the linking command line tried to link
+        against Qt5WebKitWidgets.
+        Debug builds are possible with and without framework-enabled builds
+        of Qt, but the debug versions of the Qt libraries have to be present.
+        Debug builds with a release version of Qt are not possible on Mac
+        since for debug builds qmake produces a linker command line where
+        all the Qt libraries have the "_debug" suffix, therefore if the debug
+        libraries are missing the build fails.
+
+        * Source/widgetsapi.pri:
+
 2013-04-19  Martin Robinson  <[email protected]>
 
         [GTK] JSCore.gir.in has a few problems

Modified: trunk/Source/widgetsapi.pri (148788 => 148789)


--- trunk/Source/widgetsapi.pri	2013-04-20 08:23:19 UTC (rev 148788)
+++ trunk/Source/widgetsapi.pri	2013-04-20 09:53:06 UTC (rev 148789)
@@ -73,9 +73,6 @@
 
 load(qt_module)
 
-# Allow doing a debug-only build of WebKit (not supported by Qt)
-macx:!debug_and_release:debug: TARGET = $$BASE_TARGET
-
 # Make sure the install_name of the QtWebKit library point to webkit
 force_independent:macx {
     # We do our own absolute path so that we can trick qmake into
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to