build-webkit: make ninja work with chromium linux
authordpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Apr 2012 00:29:24 +0000 (00:29 +0000)
committerdpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Apr 2012 00:29:24 +0000 (00:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=84213

Reviewed by Eric Seidel.

Fix typos so that build-webkit will use ninja to
build chromium if a build.ninja file exists and is up-to-date.

* Scripts/webkitdirs.pm:
(determineIsChromiumNinja):

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index c317365..fde84c6 100644 (file)
@@ -1,3 +1,16 @@
+2012-04-17  Dirk Pranke  <dpranke@chromium.org>
+
+        build-webkit: make ninja work with chromium linux
+        https://bugs.webkit.org/show_bug.cgi?id=84213
+
+        Reviewed by Eric Seidel.
+
+        Fix typos so that build-webkit will use ninja to
+        build chromium if a build.ninja file exists and is up-to-date.
+
+        * Scripts/webkitdirs.pm:
+        (determineIsChromiumNinja):
+
 2012-04-17  Ojan Vafai  <ojan@chromium.org>
 
         Add caching to the flakiness dashboard json files
index 8121357..b1e2512 100755 (executable)
@@ -1156,7 +1156,7 @@ sub determineIsChromiumNinja()
 
     my $hasUpToDateNinjabuild = 0;
     if (-e "out/$config/build.ninja") {
-        my $statNinja = stat("out/$config/build.ninja");
+        my $statNinja = stat("out/$config/build.ninja")->mtime;
 
         my $statXcode = 0;
         if (-e 'Source/WebKit/chromium/WebKit.xcodeproj') {
@@ -1165,7 +1165,7 @@ sub determineIsChromiumNinja()
 
         my $statMake = 0;
         if (-e 'Makefile.chromium') {
-          $statXcode = stat('Makefile.chromium')->mtime;
+          $statMake = stat('Makefile.chromium')->mtime;
         }
 
         $hasUpToDateNinjabuild = $statNinja > $statXcode && $statNinja > $statMake;