Unreviewed, rolling out r121766.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jul 2012 16:04:50 +0000 (16:04 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jul 2012 16:04:50 +0000 (16:04 +0000)
http://trac.webkit.org/changeset/121766
https://bugs.webkit.org/show_bug.cgi?id=90465

It caused flakey build errors on the bots (Requested by Ossy
on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-07-03

.:

* Source/QtWebKit.pro:
* Source/api.pri:
* Source/tests.pri:
* WebKit.pro:

Source/WebCore:

* DerivedSources.pri:
* WebCore.pri:

Source/WebKit/qt:

* declarative/declarative.pro:
* declarative/public.pri:
* tests/qgraphicswebview/qgraphicswebview.pro:

Source/WebKit2:

* DerivedSources.pri:
* Target.pri:

Source/WTF:

* WTF.pri:

Tools:

* DumpRenderTree/qt/DumpRenderTree.pro:
* QtTestBrowser/QtTestBrowser.pro:
* Scripts/webkitdirs.pm:
(qtFeatureDefaults):
(buildQMakeProjects):
* Tools.pro:
* WebKitTestRunner/InjectedBundle/DerivedSources.pri:
* WebKitTestRunner/InjectedBundle/Target.pri:
* WebKitTestRunner/Target.pri:
* qmake/.qmake.conf: Removed.
* qmake/configure.pro:
* qmake/mkspecs/features/default_post.prf:
* qmake/mkspecs/features/default_pre.prf:
* qmake/mkspecs/features/features.prf:

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

30 files changed:
ChangeLog
Source/QtWebKit.pro
Source/WTF/ChangeLog
Source/WTF/WTF.pri
Source/WebCore/ChangeLog
Source/WebCore/DerivedSources.pri
Source/WebCore/WebCore.pri
Source/WebKit/qt/ChangeLog
Source/WebKit/qt/declarative/declarative.pro
Source/WebKit/qt/declarative/public.pri
Source/WebKit/qt/tests/qgraphicswebview/qgraphicswebview.pro
Source/WebKit2/ChangeLog
Source/WebKit2/DerivedSources.pri
Source/WebKit2/Target.pri
Source/api.pri
Source/tests.pri
Tools/ChangeLog
Tools/DumpRenderTree/qt/DumpRenderTree.pro
Tools/QtTestBrowser/QtTestBrowser.pro
Tools/Scripts/webkitdirs.pm
Tools/Tools.pro
Tools/WebKitTestRunner/InjectedBundle/DerivedSources.pri
Tools/WebKitTestRunner/InjectedBundle/Target.pri
Tools/WebKitTestRunner/Target.pri
Tools/qmake/.qmake.conf [deleted file]
Tools/qmake/configure.pro
Tools/qmake/mkspecs/features/default_post.prf
Tools/qmake/mkspecs/features/default_pre.prf
Tools/qmake/mkspecs/features/features.prf
WebKit.pro

index 554d84c..9ab71e9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2012-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r121766.
+        http://trac.webkit.org/changeset/121766
+        https://bugs.webkit.org/show_bug.cgi?id=90465
+
+        It caused flakey build errors on the bots (Requested by Ossy
+        on #webkit).
+
+        * Source/QtWebKit.pro:
+        * Source/api.pri:
+        * Source/tests.pri:
+        * WebKit.pro:
+
 2012-07-03  George Staikos  <staikos@webkit.org>
 
         [BlackBerry] Enable Custom Scheme Handlers for BlackBerry.
index 1cc44a8..9d63fb8 100644 (file)
@@ -4,6 +4,8 @@
 # See 'Tools/qmake/README' for an overview of the build system
 # -------------------------------------------------------------------
 
+load(features)
+
 TEMPLATE = subdirs
 CONFIG += ordered
 
index 213213c..5a21f4e 100644 (file)
@@ -1,3 +1,14 @@
+2012-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r121766.
+        http://trac.webkit.org/changeset/121766
+        https://bugs.webkit.org/show_bug.cgi?id=90465
+
+        It caused flakey build errors on the bots (Requested by Ossy
+        on #webkit).
+
+        * WTF.pri:
+
 2012-07-03  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 
         [Qt] Make use of .qmake.cache for caching features
index 65b377c..0f99ec0 100644 (file)
@@ -5,6 +5,8 @@
 # See 'Tools/qmake/README' for an overview of the build system
 # -------------------------------------------------------------------
 
+load(features)
+
 # All external modules should include WTF headers by prefixing with "wtf" (#include <wtf/some/thing.h>).
 INCLUDEPATH += $$PWD
 
index 47b6af0..f8d5770 100644 (file)
@@ -1,3 +1,15 @@
+2012-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r121766.
+        http://trac.webkit.org/changeset/121766
+        https://bugs.webkit.org/show_bug.cgi?id=90465
+
+        It caused flakey build errors on the bots (Requested by Ossy
+        on #webkit).
+
+        * DerivedSources.pri:
+        * WebCore.pri:
+
 2012-07-03  George Staikos  <staikos@webkit.org>
 
         [BlackBerry] Enable Custom Scheme Handlers for BlackBerry.
index e90b92d..48ed801 100644 (file)
@@ -11,6 +11,8 @@
 sanitizedFile = $$toSanitizedPath($$_FILE_)
 equals(sanitizedFile, $$toSanitizedPath($$_PRO_FILE_)):TEMPLATE = derived
 
+load(features)
+
 mac {
     # FIXME: This runs the perl script every time. Is there a way we can run it only when deps change?
     fwheader_generator.commands = perl $${ROOT_WEBKIT_DIR}/Source/WebKit2/Scripts/generate-forwarding-headers.pl $${ROOT_WEBKIT_DIR}/Source/WebCore $${ROOT_BUILD_DIR}/Source/include mac
index 16aebbc..37d1332 100644 (file)
@@ -5,6 +5,8 @@
 # See 'Tools/qmake/README' for an overview of the build system
 # -------------------------------------------------------------------
 
+load(features)
+
 SOURCE_DIR = $${ROOT_WEBKIT_DIR}/Source/WebCore
 
 # We enable TextureMapper by default; remove this line to enable GraphicsLayerQt.
index 335aa2d..b93da29 100644 (file)
@@ -1,3 +1,16 @@
+2012-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r121766.
+        http://trac.webkit.org/changeset/121766
+        https://bugs.webkit.org/show_bug.cgi?id=90465
+
+        It caused flakey build errors on the bots (Requested by Ossy
+        on #webkit).
+
+        * declarative/declarative.pro:
+        * declarative/public.pri:
+        * tests/qgraphicswebview/qgraphicswebview.pro:
+
 2012-07-03  Tor Arne Vestbø  <tor.arne.vestbo@nokia.com>
 
         [Qt] Make use of .qmake.cache for caching features
index 7379283..e7ed449 100644 (file)
@@ -7,6 +7,8 @@
 TEMPLATE = subdirs
 CONFIG += ordered
 
+load(features)
+
 public_api.file = public.pri
 public_api.makefile = Makefile.declarative.public
 SUBDIRS += public_api
index a3feb7d..576ce03 100644 (file)
@@ -11,6 +11,8 @@ TARGET.module_name = QtWebKit
 
 CONFIG += qt plugin
 
+load(features)
+
 QMLDIRFILE = $${_PRO_FILE_PWD_}/qmldir
 copy2build.input = QMLDIRFILE
 copy2build.output = $${ROOT_BUILD_DIR}/imports/$${TARGET.module_name}/qmldir
index 3e9fb15..1e0ab0a 100644 (file)
@@ -1,6 +1,7 @@
 include(../tests.pri)
 exists($${TARGET}.qrc):RESOURCES += $${TARGET}.qrc
 
+load(features)
 contains(DEFINES, ENABLE_WEBGL=1) {
     QT += opengl
 }
index 44ede1b..0d3921d 100644 (file)
@@ -1,3 +1,15 @@
+2012-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r121766.
+        http://trac.webkit.org/changeset/121766
+        https://bugs.webkit.org/show_bug.cgi?id=90465
+
+        It caused flakey build errors on the bots (Requested by Ossy
+        on #webkit).
+
+        * DerivedSources.pri:
+        * Target.pri:
+
 2012-07-03  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Add WebKitWebView::context-menu-dismissed signal to WebKit2 GTK+ API
index ab42707..319bf7e 100644 (file)
@@ -11,6 +11,8 @@
 sanitizedFile = $$toSanitizedPath($$_FILE_)
 equals(sanitizedFile, $$toSanitizedPath($$_PRO_FILE_)):TEMPLATE = derived
 
+load(features)
+
 WEBCORE_GENERATED_SOURCES_DIR = ../WebCore/$${GENERATED_SOURCES_DESTDIR}
 
 SOURCE_DIR = $${ROOT_WEBKIT_DIR}/Source
index 6c2dcb5..4e86026 100644 (file)
@@ -7,6 +7,8 @@
 TEMPLATE = lib
 TARGET = WebKit2
 
+load(features)
+
 include(WebKit2.pri)
 
 WEBKIT += wtf javascriptcore webcore
index 2823b56..7ae29fc 100644 (file)
@@ -124,6 +124,8 @@ haveQt(5) {
 
 runSyncQt() # Generate forwarding headers for the QtWebKit API
 
+load(features)
+
 WEBKIT += wtf
 
 !v8:WEBKIT += javascriptcore
index 0042fc5..2781578 100644 (file)
@@ -7,6 +7,8 @@
 TEMPLATE = subdirs
 CONFIG += ordered
 
+load(features)
+
 WEBKIT_TESTS_DIR = $$PWD/WebKit/qt/tests
 
 SUBDIRS += \
index 110d4b0..625729e 100644 (file)
@@ -1,3 +1,27 @@
+2012-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r121766.
+        http://trac.webkit.org/changeset/121766
+        https://bugs.webkit.org/show_bug.cgi?id=90465
+
+        It caused flakey build errors on the bots (Requested by Ossy
+        on #webkit).
+
+        * DumpRenderTree/qt/DumpRenderTree.pro:
+        * QtTestBrowser/QtTestBrowser.pro:
+        * Scripts/webkitdirs.pm:
+        (qtFeatureDefaults):
+        (buildQMakeProjects):
+        * Tools.pro:
+        * WebKitTestRunner/InjectedBundle/DerivedSources.pri:
+        * WebKitTestRunner/InjectedBundle/Target.pri:
+        * WebKitTestRunner/Target.pri:
+        * qmake/.qmake.conf: Removed.
+        * qmake/configure.pro:
+        * qmake/mkspecs/features/default_post.prf:
+        * qmake/mkspecs/features/default_pre.prf:
+        * qmake/mkspecs/features/features.prf:
+
 2012-07-03  George Staikos  <staikos@webkit.org>
 
         [BlackBerry] Enable Custom Scheme Handlers for BlackBerry.
index f47fb43..327dc3a 100644 (file)
@@ -9,6 +9,8 @@ TEMPLATE = app
 TARGET = DumpRenderTree
 DESTDIR = $$ROOT_BUILD_DIR/bin
 
+load(features)
+
 WEBKIT += wtf webcore
 !v8: WEBKIT += javascriptcore
 
index 6c9352c..3bff490 100644 (file)
@@ -11,6 +11,8 @@ INCLUDEPATH += \
     $${ROOT_WEBKIT_DIR}/Tools/DumpRenderTree/qt/ \
     $${ROOT_WEBKIT_DIR}/Source/WTF
 
+load(features)
+
 SOURCES += \
     $${ROOT_WEBKIT_DIR}/Tools/DumpRenderTree/qt/QtInitializeTestFonts.cpp \
     locationedit.cpp \
index 9e0b7a1..c1fb8ee 100755 (executable)
@@ -824,40 +824,24 @@ sub qtFeatureDefaults
 
     my $file;
     my @buildArgs;
-    my $qconfigs;
 
     if (@_) {
         @buildArgs = (@buildArgs, @{$_[0]});
-        $qconfigs = $_[1];
         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";
     }
 
-    my @defaults = `$qmakecommand @buildArgs $file 2>&1`;
+    my $defaults = `$qmakecommand @buildArgs $file 2>&1`;
 
     my %qtFeatureDefaults;
-    for (@defaults) {
-        if (/ DEFINES: /) {
-            while (/(\S+?)=(\S+?)/gi) {
-                $qtFeatureDefaults{$1}=$2;
-            }
-        } elsif (/ CONFIG:(.*)$/) {
-            if (@_) {
-                $$qconfigs = $1;
-            }
-        } elsif (/Done computing defaults/) {
-            print "\n";
-            last;
-        } elsif (@_) {
-            print $_;
-        }
+    while ($defaults =~ m/(\S+?)=(\S+?)/gi) {
+        $qtFeatureDefaults{$1}=$2;
     }
 
     chdir $originalCwd;
@@ -2246,7 +2230,6 @@ sub buildQMakeProjects
     my ($projects, $clean, @buildParams) = @_;
 
     my @buildArgs = ();
-    my $qconfigs = "";
 
     my $make = qtMakeCommand($qmakebin);
     my $makeargs = "";
@@ -2303,7 +2286,7 @@ sub buildQMakeProjects
     File::Path::mkpath($dir);
     chdir $dir or die "Failed to cd into " . $dir . "\n";
 
-    my %defines = qtFeatureDefaults(\@buildArgs, \$qconfigs);
+    my %defines = qtFeatureDefaults(\@buildArgs);
 
     my $svnRevision = currentSVNRevision();
 
@@ -2312,8 +2295,6 @@ sub buildQMakeProjects
     my $pathToDefinesCache = File::Spec->catfile($dir, ".webkit.config");
     my $pathToOldDefinesFile = File::Spec->catfile($dir, "defaults.txt");
 
-    # FIXME: Get rid of .webkit.config and defaults.txt and move all the logic to .qmake.cache
-
     # Ease transition to new build layout
     if (-e $pathToOldDefinesFile) {
         print "Old build layout detected";
@@ -2376,24 +2357,13 @@ sub buildQMakeProjects
 
             # After removing WebKitBuild directory, we have to call qtFeatureDefaults()
             # to run config tests and generate the removed Tools/qmake/.qmake.cache again.
-            qtFeatureDefaults(\@buildArgs, \$qconfigs);
+            qtFeatureDefaults(\@buildArgs);
         #}
 
         # Still trigger an incremental build
         $buildHint = "incremental";
     }
 
-    if ($buildHint eq "incremental") {
-        my $qmakeDefines = "DEFINES +=";
-        foreach my $key (sort keys %defines) {
-            $qmakeDefines .= " \\\n    $key=$defines{$key}";
-        }
-        open(QMAKE_CACHE, ">.qmake.cache") or die "Cannot create .qmake.cache!\n";
-        print QMAKE_CACHE "CONFIG += webkit_configured $qconfigs\n";
-        print QMAKE_CACHE $qmakeDefines."\n";
-        close(QMAKE_CACHE);
-    }
-
     # Save config up-front so we can detect changes to the build config even
     # when the user re-configures after aborting the build.
     open(DEFAULTS, ">$pathToDefinesCache");
index 5b7b5de..7f97eff 100644 (file)
@@ -7,6 +7,8 @@
 TEMPLATE = subdirs
 CONFIG += ordered
 
+load(features)
+
 !no_webkit1 {
     SUBDIRS += QtTestBrowser/QtTestBrowser.pro
     SUBDIRS += DumpRenderTree/qt/DumpRenderTree.pro
index 07cdc67..651ad22 100644 (file)
@@ -11,6 +11,8 @@
 sanitizedFile = $$toSanitizedPath($$_FILE_)
 equals(sanitizedFile, $$toSanitizedPath($$_PRO_FILE_)):TEMPLATE = derived
 
+load(features)
+
 IDL_BINDINGS += \
     Bindings/AccessibilityController.idl \
     Bindings/AccessibilityTextMarker.idl \
index 0374352..7d382ac 100644 (file)
@@ -52,6 +52,8 @@ DESTDIR = $${ROOT_BUILD_DIR}/lib
 
 QT += declarative widgets webkit
 
+load(features)
+
 WEBKIT += wtf javascriptcore webcore
 
 CONFIG += plugin rpath
index e46906c..b9e7a8b 100644 (file)
@@ -27,6 +27,8 @@ DESTDIR = $${ROOT_BUILD_DIR}/bin
 
 QT = core gui widgets network declarative testlib quick quick-private webkit
 
+load(features)
+
 WEBKIT += wtf javascriptcore webkit2
 
 DEFINES += USE_SYSTEM_MALLOC=1
diff --git a/Tools/qmake/.qmake.conf b/Tools/qmake/.qmake.conf
deleted file mode 100644 (file)
index 2f0dc89..0000000
+++ /dev/null
@@ -1,2 +0,0 @@
-# This file is only here to isolate the configure project
-# from the rest of the tree.
index c68f54a..2da6a1b 100644 (file)
@@ -1,18 +1,3 @@
-# -------------------------------------------------------------------
-# This file is used by build-webkit to compute the various feature
-# defines, which are then cached in .qmake.cache.
-#
-# See 'Tools/qmake/README' for an overview of the build system
-# -------------------------------------------------------------------
-
-# Will compute features based on command line arguments, config tests,
-# dependency availability, and defaults.
 load(features)
-
-# Compute delta
-CONFIG -= $$BASE_CONFIG
-DEFINES -= $$BASE_DEFINES
-
-message(CONFIG: $$CONFIG)
-message(DEFINES: $$DEFINES)
+message($$DEFINES)
 error("Done computing defaults")
index 4e7d2c4..56c109c 100644 (file)
@@ -5,6 +5,8 @@
 # See 'Tools/qmake/README' for an overview of the build system
 # -------------------------------------------------------------------
 
+load(features)
+
 # Flag that we're now done processing the project file. This allows
 # a feature to distinguish between being processed due to a load()
 # or include() and a CONFIG += feature.
index 25593e7..bb82e15 100644 (file)
@@ -27,13 +27,10 @@ ROOT_WEBKIT_DIR = $$replace(IN_PWD, /Tools/qmake/mkspecs/features$,)
 WEBKIT_SUBDIR = $$replace(_PRO_FILE_PWD_, $${ROOT_WEBKIT_DIR},)
 ROOT_BUILD_DIR = $$replace(OUT_PWD, $${WEBKIT_SUBDIR}$,)
 
-# Detect if we're running the initial configure step
-pro_file_name = $$basename(_PRO_FILE_)
-equals(pro_file_name, configure.pro): CONFIG += configure_pass
-
 # We want the QtWebKit API forwarding includes to live in the root build dir,
 # except when we are running the config.tests in Tools/qmake.
-!configure_pass: QMAKE_SYNCQT_OUTDIR = $$ROOT_BUILD_DIR
+pro_file_name = $$basename(_PRO_FILE_)
+!equals(pro_file_name, configure.pro): QMAKE_SYNCQT_OUTDIR = $$ROOT_BUILD_DIR
 
 # Load Qt's defaults after we've resolved the build directory. This will also
 # run syncqt if there is a sync.profile in the $$_PRO_FILE_PWD_. We never want
@@ -126,30 +123,3 @@ haveQt(4) {
 
 # Set some defaults for specific platforms
 CONFIG += include_webinspector
-
-# Try to locate sqlite3 source (for WebCore)
-SQLITE3SRCDIR = $$(SQLITE3SRCDIR)
-isEmpty(SQLITE3SRCDIR) {
-    haveQt(5): SQLITE3SRCDIR = $$QT.core.sources/../3rdparty/sqlite/
-    else: SQLITE3SRCDIR = $$[QT_INSTALL_PREFIX]/src/3rdparty/sqlite/
-}
-
-# --------- Make features available to JSC ------------
-
-## Forward enabled feature macros to JavaScript enabled features macros
-FEATURE_DEFINES_JAVASCRIPT = LANGUAGE_JAVASCRIPT=1
-v8: FEATURE_DEFINES_JAVASCRIPT += V8_BINDING=1
-
-for(define, DEFINES) {
-    enabled_feature_define = $$find(define, ^ENABLE_.+=1$)
-    isEmpty(enabled_feature_define): next()
-
-    FEATURE_DEFINES_JAVASCRIPT += $$enabled_feature_define
-}
-
-configure_pass {
-    # Save the base config and defines so we can compute the delta between
-    # what features.prf introduced and what was already there.
-    BASE_CONFIG = $$CONFIG
-    BASE_DEFINES = $$DEFINES
-}
index 6d99d11..1e6950a 100644 (file)
 ## load mobilityconfig if mobility is available
 load(mobilityconfig, true)
 
+# Try to locate sqlite3 source
+SQLITE3SRCDIR = $$(SQLITE3SRCDIR)
+isEmpty(SQLITE3SRCDIR) {
+    haveQt(5):SQLITE3SRCDIR = $$QT.core.sources/../3rdparty/sqlite/
+    else:SQLITE3SRCDIR = $$[QT_INSTALL_PREFIX]/src/3rdparty/sqlite/
+}
+
 # ---------- Dynamically detect optional features -------------
 #
 # Please note, static feature defaults go in features.pri
@@ -213,13 +220,24 @@ for(define, DEFINES) {
     contains(DEFINES, $$opposite): DEFINES -= $$define
 }
 
+# --------- Make features available to JSC ------------
+
+## Forward enabled feature macros to JavaScript enabled features macros
+FEATURE_DEFINES_JAVASCRIPT = LANGUAGE_JAVASCRIPT=1
+v8: FEATURE_DEFINES_JAVASCRIPT += V8_BINDING=1
+
+for(define, DEFINES) {
+    enabled_feature_define = $$find(define, ^ENABLE_.+=1$)
+    isEmpty(enabled_feature_define): next()
+
+    FEATURE_DEFINES_JAVASCRIPT += $$enabled_feature_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)
+    message($$DEFINES)
     error("Done computing defaults")
 }
index 2b2aff6..9b4007b 100644 (file)
@@ -8,6 +8,8 @@
 TEMPLATE = subdirs
 CONFIG += ordered
 
+load(features)
+
 QMAKEPATH = $$(QMAKEPATH)
 isEmpty(QMAKEPATH)|!exists($${QMAKEPATH}/mkspecs) {
     error("The environment variable QMAKEPATH needs to point to $WEBKITSRC/Tools/qmake")