[Qt] Ensure forwarding headers are generated before generating install rules
authorvestbo@webkit.org <vestbo@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Nov 2011 11:30:47 +0000 (11:30 +0000)
committervestbo@webkit.org <vestbo@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Nov 2011 11:30:47 +0000 (11:30 +0000)
The install rule for headers depends on qmake being able to walk the list of
headers in the $build_root/include/QtWebKit directory, but this directory is
empty until we've generated the forwarding headers. So we need to make sure
the forwarding headers are generated _before_ we run qmake on the api project
file. This applies to Qt 4.8 only, as Qt 5 will run synqt as part of the
root project file (due to the sync.profile file).

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

Reviewed by Simon Hausmann.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@99542 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Source/QtWebKit.pro
Source/api.pri

index 16791ab685986d0b45e4e0de7286428f641b71bd..25707b87659889e1801da85e634a097cfa6d7ead 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,21 @@
+2011-11-08  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
+
+        [Qt] Ensure forwarding headers are generated before generating install rules
+
+        The install rule for headers depends on qmake being able to walk the list of
+        headers in the $build_root/include/QtWebKit directory, but this directory is
+        empty until we've generated the forwarding headers. So we need to make sure
+        the forwarding headers are generated _before_ we run qmake on the api project
+        file. This applies to Qt 4.8 only, as Qt 5 will run synqt as part of the
+        root project file (due to the sync.profile file).
+
+        https://bugs.webkit.org/show_bug.cgi?id=71697
+
+        Reviewed by Simon Hausmann.
+
+        * Source/QtWebKit.pro:
+        * Source/api.pri:
+
 2011-11-07  ChangSeok Oh  <shivamidow@gmail.com>
 
         [EFL] Support requestAnimationFrame API
index 34ee5223d99be679d651438f0e09fafecc125399..9c861d5e8630c7c9afa627ed6ad03c23d4066660 100644 (file)
@@ -26,3 +26,26 @@ examples.file = WebKit/qt/examples/examples.pro
 examples.CONFIG += no_default_target
 examples.makefile = Makefile
 SUBDIRS += examples
+
+!haveQt(5):!build_pass {
+    # Use our own copy of syncqt from Qt 4.8 to generate forwarding headers
+    syncqt = $$toSystemPath($${ROOT_WEBKIT_DIR}/Tools/qmake/syncqt-4.8)
+    command = $$syncqt
+    win32-msvc*: command = $$command -windows
+
+    outdir = $$toSystemPath($${ROOT_BUILD_DIR})
+    rootdir = $$toSystemPath($${ROOT_WEBKIT_DIR})
+
+    fwheader_generator.commands = perl $${command} -outdir $${outdir} -separate-module $${TARGET}$${DIRLIST_SEPARATOR}$${rootdir}$${DIRLIST_SEPARATOR}$$toSystemPath(Source/WebKit/qt/Api)
+    fwheader_generator.depends = $${syncqt}
+
+    variables = $$computeSubdirVariables(api)
+
+    api_qmake.target = $$eval($${variables}.target)-qmake_all
+    api_qmake.depends = fwheader_generator
+
+    api_makefile.target = $$eval($${variables}.makefile)
+    api_makefile.depends = fwheader_generator
+
+    QMAKE_EXTRA_TARGETS += fwheader_generator api_qmake api_makefile
+}
index 59990d3af5948c5876aa0e7ab96a55c203bde0f3..b2eddb43b82d9b13d20dab6a1f1b9c73759f670f 100644 (file)
@@ -43,21 +43,6 @@ VERSION = $${QT.webkit.MAJOR_VERSION}.$${QT.webkit.MINOR_VERSION}.$${QT.webkit.P
 
 !static: DEFINES += QT_MAKEDLL
 
-!haveQt(5):!build_pass {
-    # Use our own copy of syncqt from Qt 4.8 to generate forwarding headers
-    syncqt = $$toSystemPath($${ROOT_WEBKIT_DIR}/Tools/qmake/syncqt-4.8)
-    command = $$syncqt
-    win32-msvc*: command = $$command -windows
-
-    outdir = $$toSystemPath($${ROOT_BUILD_DIR})
-    rootdir = $$toSystemPath($${ROOT_WEBKIT_DIR})
-
-    fwheader_generator.commands = perl $${command} -outdir $${outdir} -separate-module $${TARGET}$${DIRLIST_SEPARATOR}$${rootdir}$${DIRLIST_SEPARATOR}$$toSystemPath(Source/WebKit/qt/Api)
-    fwheader_generator.depends = $${syncqt}
-    QMAKE_EXTRA_TARGETS += fwheader_generator
-    DEFAULT_TARGETS += fwheader_generator
-}
-
 SOURCES += \
     $$PWD/WebKit/qt/WebCoreSupport/QtFallbackWebPopup.cpp \
     $$PWD/WebKit/qt/WebCoreSupport/QtWebComboBox.cpp \