[Qt] Unify qtFeatureDefaults code paths
authorossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2012 13:16:33 +0000 (13:16 +0000)
committerossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 6 Jul 2012 13:16:33 +0000 (13:16 +0000)
There's no reason to run qmake on features.prf directly anymore, as we
can selectivly run configure tests in the project file now.

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

Patch by Oswald Buddenhagen <oswald.buddenhagen@nokia.com> on 2012-07-06
Reviewed by Tor Arne Vestbø.

* Scripts/webkitdirs.pm:
(qtFeatureDefaults):
* qmake/configure.pri:
* qmake/mkspecs/features/features.prf:

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm
Tools/qmake/configure.pri
Tools/qmake/mkspecs/features/features.prf

index b06b967e00c57a74a5ddc754210938ec40bc1e2c..952f8e65cd2381d2b35956fd2c1d245366671eac 100644 (file)
@@ -1,3 +1,19 @@
+2012-07-06  Oswald Buddenhagen  <oswald.buddenhagen@nokia.com>
+
+        [Qt] Unify qtFeatureDefaults code paths
+
+        There's no reason to run qmake on features.prf directly anymore, as we
+        can selectivly run configure tests in the project file now.
+
+        https://bugs.webkit.org/show_bug.cgi?id=90461
+
+        Reviewed by Tor Arne Vestbø.
+
+        * Scripts/webkitdirs.pm:
+        (qtFeatureDefaults):
+        * qmake/configure.pri:
+        * qmake/mkspecs/features/features.prf:
+
 2012-07-06  Oswald Buddenhagen  <oswald.buddenhagen@nokia.com>
 
         [Qt] Don't let qt_webkit.pri proclaim its own location
index 26d8bc3b177f9449e900a9f3e3437ead02627a87..b1d29104debd5932dfaa29a0c3d1412d2095ece3 100755 (executable)
@@ -822,7 +822,7 @@ sub qtFeatureDefaults
 
     my $originalCwd = getcwd();
 
-    my $file;
+    my $file = File::Spec->catfile($qmakepath, "configure.pro");
     my @buildArgs;
     my $qconfigs;
 
@@ -832,12 +832,9 @@ sub qtFeatureDefaults
         my $dir = File::Spec->catfile(productDir(), "Tools", "qmake");
         File::Path::mkpath($dir);
         chdir $dir or die "Failed to cd into " . $dir . "\n";
-        $file = File::Spec->catfile($qmakepath, "configure.pro");
     } else {
         # Do a quick check of the features without running the config tests
-        # FIXME: When Qt supports it, go through configure.pro but without config tests
-        $file = File::Spec->catfile($qmakepath, "mkspecs", "features", "features.prf");
-        push @buildArgs, "CONFIG+=compute_defaults";
+        push @buildArgs, "CONFIG+=quick_check";
     }
 
     my @defaults = `$qmakecommand @buildArgs -nocache $file 2>&1`;
index 451fe3fefcb1b1d49e9558844361101674320db9..8c009fd20abd17786ba0f0cb81e259a1e026ae45 100644 (file)
@@ -5,7 +5,7 @@
 # See 'Tools/qmake/README' for an overview of the build system
 # -------------------------------------------------------------------
 
-haveQt(5) {
+haveQt(5):!quick_check {
     load(configure)
     QMAKE_CONFIG_TESTS_DIR = $$PWD/config.tests
 
index 68849c044157a22a12486aca852a0cc497beec82..e76c8090e86d3f17b681c38c34f43ffea79cf000 100644 (file)
@@ -212,14 +212,3 @@ for(define, DEFINES) {
     # Is there a later define that overrides this one?
     contains(DEFINES, $$opposite): DEFINES -= $$define
 }
-
-# ---------------------- The end ----------------------
-
-# Used to compute defaults for the build-webkit script
-# Don't place anything after this!
-CONFIG(compute_defaults) {
-    # FIXME: We should have configure.pro do this when the config tests
-    # can be selectivly run.
-    message(DEFINES: $$DEFINES)
-    error("Done computing defaults")
-}