Title: [131446] trunk/Tools
Revision
131446
Author
[email protected]
Date
2012-10-16 05:53:21 -0700 (Tue, 16 Oct 2012)

Log Message

Fix build-webkit bailing out of !isSVN() and !isGit()

Reviewed by Tor Arne Vestbø.

Added missing else case with early return.

* Scripts/VCSUtils.pm:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (131445 => 131446)


--- trunk/Tools/ChangeLog	2012-10-16 12:48:32 UTC (rev 131445)
+++ trunk/Tools/ChangeLog	2012-10-16 12:53:21 UTC (rev 131446)
@@ -1,3 +1,13 @@
+2012-10-16  Simon Hausmann  <[email protected]>
+
+        Fix build-webkit bailing out of !isSVN() and !isGit()
+
+        Reviewed by Tor Arne Vestbø.
+
+        Added missing else case with early return.
+
+        * Scripts/VCSUtils.pm:
+
 2012-10-16  Simon Hausmann  <[email protected]>, Tor Arne Vestbø <[email protected]>
 
         [Qt] Add logic for triggering clean builds on changes to build system files

Modified: trunk/Tools/Scripts/VCSUtils.pm (131445 => 131446)


--- trunk/Tools/Scripts/VCSUtils.pm	2012-10-16 12:48:32 UTC (rev 131445)
+++ trunk/Tools/Scripts/VCSUtils.pm	2012-10-16 12:53:21 UTC (rev 131446)
@@ -2120,6 +2120,8 @@
         $command = "git diff --name-status $firstCommit..$lastCommit";
     } elsif (isSVN()) {
         $command = "svn diff --summarize -r $firstRevision:$lastRevision";
+    } else {
+        return ();
     }
     my $diffOutput = `$command`;
     $diffOutput =~ s/^[A-Z]\s+//gm;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to