2006-10-03 Nikolas Zimmermann <zimmermann@kde.org>
authoreseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Oct 2006 12:14:44 +0000 (12:14 +0000)
committereseidel <eseidel@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Oct 2006 12:14:44 +0000 (12:14 +0000)
        Reviewed by eseidel.  Landed by eseidel.

        Offer a way for BuildBot to not colorize the cmake output, when building
        with the 'build-webkit' script. Add "--color" / "--no-color" option pair.

        Default is colorize output, though only Qt platform handles this for now.

        * Scripts/build-dumprendertree:
        * Scripts/build-webkit:
        * Scripts/webkitdirs.pm:

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

WebKitTools/ChangeLog
WebKitTools/Scripts/build-dumprendertree
WebKitTools/Scripts/build-webkit
WebKitTools/Scripts/webkitdirs.pm

index 013b67a67e9c3a1bda6865511852d390908d4a2b..a75ddfeb5abce243aac7d4b126f61372a42d758d 100644 (file)
@@ -1,3 +1,16 @@
+2006-10-03  Nikolas Zimmermann  <zimmermann@kde.org>
+
+        Reviewed by eseidel.  Landed by eseidel.
+
+        Offer a way for BuildBot to not colorize the cmake output, when building
+        with the 'build-webkit' script. Add "--color" / "--no-color" option pair.
+
+        Default is colorize output, though only Qt platform handles this for now.
+
+        * Scripts/build-dumprendertree:
+        * Scripts/build-webkit:
+        * Scripts/webkitdirs.pm:
+
 2006-10-03  Nikolas Zimmermann  <zimmermann@kde.org>
 
         Reviewed by eseidel.  Landed by eseidel.
index b63d1ecdff1656a88d064dfa3ef9f2394c46030f..702155b41f2eec4229d8b72414028515444424ce 100755 (executable)
@@ -50,7 +50,7 @@ if (isOSX()) {
     $result = buildVisualStudioProject("DumpRenderTree");
 } elsif (isQt()) {
     # Qt/Linux builds everything in one-shot.
-    $result = buildCMakeProject("WebKit");
+    $result = buildCMakeProject("WebKit", 1);
 } else {
     die "Building not defined for this platform!\n";
 }
index 7567e3fa9141b16e4c15b36a31e1c3d48ccbabd5..ae85ccce99207497bb63391e65f77bff75a146cb 100755 (executable)
@@ -36,11 +36,15 @@ use webkitdirs;
 use POSIX;
 
 my $svgSupport = 1;
-GetOptions('svg!' => \$svgSupport);
+my $colorSupport = 1; # Default: colorize make output
+
+GetOptions('svg!' => \$svgSupport,
+           'color!' => \$colorSupport);
 
 checkRequiredSystemConfig();
 setConfiguration();
 chdirWebKit();
+
 my $productDir = productDir();
 my @options = XcodeOptions();
 my @additionalWebCoreOptions = ();
@@ -96,7 +100,7 @@ for my $dir (@projects) {
         }
     } elsif (isQt()) {
         if ($dir eq "WebKit") {
-            $result = buildCMakeProject($dir);
+            $result = buildCMakeProject($dir, $colorSupport);
         }
     }
 
index cdec9ce592a2506e7b2c81864dbd1a748bd5b83b..f31b60566d202cbe2d4ee49a613bf74611ff57d1 100644 (file)
@@ -388,21 +388,26 @@ sub buildVisualStudioProject($)
     return $result;
 }
 
-sub buildCMakeProject($)
+sub buildCMakeProject($$)
 {
-    my ($project) = @_;
+    my ($project, $colorize) = @_;
 
     if ($project ne "WebKit") { 
         die "Qt/Linux builds JavaScriptCore/WebCore/WebKitQt in one shot! Only call it for 'WebKit'.\n";
     }
 
     my $config = configuration();
-    print "Calling 'cmake -DCMAKE_BUILD_TYPE=$config ../' in " . baseProductDir() . "/$config ...\n\n";
+    my @buildArgs = ("-DCMAKE_BUILD_TYPE=$config");
+    push @buildArgs, "-DWEBKIT_DO_NOT_USE_COLORFUL_OUTPUT=" . ($colorize ? "OFF" : "ON");
+    push @buildArgs, "../../";
+
+    print "Calling 'cmake @buildArgs' in " . baseProductDir() . "/$config ...\n\n";
 
     system "mkdir -p " . baseProductDir() . "/$config";
     chdir baseProductDir() . "/$config" or die "Failed to cd into " . baseProductDir() . "/$config \n";
 
-    my $result = system "cmake", "-DCMAKE_BUILD_TYPE=$config", "../../";
+    my $result = system "cmake", @buildArgs;
     if($result ne 0) {
        die "Failed to setup build environment using cmake!\n";
     }