kill-old-processes spits out error messages on Mac
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Mar 2013 19:19:23 +0000 (19:19 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Mar 2013 19:19:23 +0000 (19:19 +0000)
https://bugs.webkit.org/show_bug.cgi?id=112878

Reviewed by Jessie Berlin.

Use grep -E instead of grep -P since grep on Mac doesn't seem to support the latter even though it's
included in the list of options grep --help shows.

Also use double-slash in escaping ++ in clang++ since the slashes needs to escape + inside the shell.

* BuildSlaveSupport/kill-old-processes:
(main):

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

Tools/BuildSlaveSupport/kill-old-processes
Tools/ChangeLog

index 272d2d9..359967f 100755 (executable)
@@ -59,7 +59,7 @@ def main():
         "cc1plus",
         "cc1objplus",
         "clang",
-        r"clang\+\+",
+        r"clang\\+\\+",
         "gcc-4.2",
         "httpd",
         "i686-apple-darwin10-gcc-4.2.1",
@@ -102,7 +102,7 @@ def main():
         for task in tasksToKillMac:
             os.system("killall -9 -v -m " + task)
         # Kill all instances of python executing run-webkit-tests
-        os.system("ps aux | grep -P '.+/Python .+(run_webkit_tests|run-webkit-tests)' | grep -v grep | awk '{print $2}' | xargs kill")
+        os.system("ps aux | grep -E '.+/Python .+(run_webkit_tests|run-webkit-tests)' | grep -v grep | awk '{print $2}' | xargs kill")
     elif sys.platform == 'cygwin' or sys.platform == 'win32':
         for task in tasksToKillWin:
             os.system("taskkill /t /f /im " + task)
index 15c9d10..338ab61 100644 (file)
@@ -1,3 +1,18 @@
+2013-03-20  Ryosuke Niwa  <rniwa@webkit.org>
+
+        kill-old-processes spits out error messages on Mac
+        https://bugs.webkit.org/show_bug.cgi?id=112878
+
+        Reviewed by Jessie Berlin.
+
+        Use grep -E instead of grep -P since grep on Mac doesn't seem to support the latter even though it's
+        included in the list of options grep --help shows.
+
+        Also use double-slash in escaping ++ in clang++ since the slashes needs to escape + inside the shell.
+
+        * BuildSlaveSupport/kill-old-processes:
+        (main):
+
 2013-03-21  Alberto Garcia  <agarcia@igalia.com>
 
         [BlackBerry] AccessibilityUIElement: add dummy implementations of scroll methods