[chromium] Fix exit code status logic in run-chromium-webkit-unit-tests
authorjamesr@google.com <jamesr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Aug 2012 19:16:22 +0000 (19:16 +0000)
committerjamesr@google.com <jamesr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 27 Aug 2012 19:16:22 +0000 (19:16 +0000)
https://bugs.webkit.org/show_bug.cgi?id=95115

Reviewed by Adam Barth.

Use VCSUtils utility function to check if webkit_unit_tests terminates normally so we generate a non-zero exit
code if the unit tests crash.

* Scripts/run-chromium-webkit-unit-tests:

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

Tools/ChangeLog
Tools/Scripts/run-chromium-webkit-unit-tests

index aae3d7c..76190f8 100644 (file)
@@ -1,3 +1,15 @@
+2012-08-27  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Fix exit code status logic in run-chromium-webkit-unit-tests
+        https://bugs.webkit.org/show_bug.cgi?id=95115
+
+        Reviewed by Adam Barth.
+
+        Use VCSUtils utility function to check if webkit_unit_tests terminates normally so we generate a non-zero exit
+        code if the unit tests crash.
+
+        * Scripts/run-chromium-webkit-unit-tests:
+
 2012-08-27  Benjamin Poulain  <benjamin@webkit.org>
 
         Add ECMAScript Number to String conversion to WTF::String
index 4e6a502..43f85ad 100755 (executable)
@@ -32,9 +32,9 @@ use File::Spec;
 use FindBin;
 use lib $FindBin::Bin;
 use webkitdirs;
+use VCSUtils;
 
 push(@ARGV, "--chromium");
 setConfiguration();
 my $pathToBinary = productDir() . "/webkit_unit_tests";
-# perl's "system" function's return value has to be right shifted by 8 to get the program's exit code.
-exit system ($pathToBinary, @ARGV) >> 8;
+exit exitStatus(system ($pathToBinary, @ARGV));