Unreviewed, rolling out r137371.
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Dec 2012 23:45:29 +0000 (23:45 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Dec 2012 23:45:29 +0000 (23:45 +0000)
http://trac.webkit.org/changeset/137371
https://bugs.webkit.org/show_bug.cgi?id=104434

Various scripts are not ready for out/ as the build directory,
this can't work as written.

* Scripts/update-webkit:
* Scripts/webkitdirs.pm:
(buildChromium):

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

Tools/ChangeLog
Tools/Scripts/update-webkit
Tools/Scripts/webkitdirs.pm

index ebc05d3..95f0c60 100644 (file)
@@ -1,3 +1,16 @@
+2012-12-11  Eric Seidel  <eric@webkit.org>
+
+        Unreviewed, rolling out r137371.
+        http://trac.webkit.org/changeset/137371
+        https://bugs.webkit.org/show_bug.cgi?id=104434
+
+        Various scripts are not ready for out/ as the build directory,
+        this can't work as written.
+
+        * Scripts/update-webkit:
+        * Scripts/webkitdirs.pm:
+        (buildChromium):
+
 2012-12-11  Zan Dobersek  <zandobersek@gmail.com>
 
         Unreviewed GTK build functionality fix.
index 92493b5..6ba8044 100755 (executable)
@@ -47,7 +47,7 @@ my $quiet = '';
 my $showHelp;
 my $useGYP = 0;
 my $useMake = 0;
-my $useNinja = 1;
+my $useNinja = 0;
 
 determineIsChromium();
 determineIsChromiumAndroid();
@@ -61,7 +61,7 @@ my $getOptionsResult = GetOptions(
     'q|quiet' => \$quiet,
     'gyp' => \$useGYP,
     'make' => \$useMake,
-    'ninja!' => \$useNinja,
+    'ninja' => \$useNinja,
 ); 
 
 if (!$getOptionsResult || $showHelp) {
@@ -69,7 +69,7 @@ if (!$getOptionsResult || $showHelp) {
 Usage: @{[ basename($0) ]} [options]
   --chromium          also update dependencies of the chromium port
   --make              generate the Makefile-based build system (Chromium only)
-  --no-ninja          generate the platform-native build system instead of ninja (Chromium only)
+  --ninja             generate the ninja-based build system (Chromium only)
   --chromium-android  also update dependencies of the chromium port for Android
   -h|--help           show the help message
   -q|--quiet          pass -q to svn update for quiet updates
index 202057f..0f9abe7 100755 (executable)
@@ -2545,7 +2545,7 @@ sub buildChromium($@)
     } elsif (isCygwin() || isWindows()) {
         # Windows build - builds the root visual studio solution.
         $result = buildChromiumVisualStudioProject("Source/WebKit/chromium/All.sln", $clean);
-    } elsif (isChromiumNinja()) {
+    } elsif (isChromiumNinja() && !isChromiumAndroid()) {
         $result = buildChromiumNinja("all", $clean, @options);
     } elsif (isLinux() || isChromiumAndroid() || isChromiumMacMake()) {
         # Linux build - build using make.