2009-06-25 Eric Seidel <eric@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 25 Jun 2009 07:42:33 +0000 (07:42 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 25 Jun 2009 07:42:33 +0000 (07:42 +0000)
        Reviewed by Tor Arne Vestbø.

        Remove use of os.system to fix reviewers with unicode chars in their names
        https://bugs.webkit.org/show_bug.cgi?id=26713

        Also change to latin1 encoding of ø to make python happy.

        * Scripts/modules/bugzilla.py:
        * Scripts/modules/scm.py:

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

WebKitTools/ChangeLog
WebKitTools/Scripts/modules/bugzilla.py
WebKitTools/Scripts/modules/scm.py

index b0dc481..935041e 100644 (file)
@@ -2,6 +2,18 @@
 
         Reviewed by Tor Arne Vestbø.
 
 
         Reviewed by Tor Arne Vestbø.
 
+        Remove use of os.system to fix reviewers with unicode chars in their names
+        https://bugs.webkit.org/show_bug.cgi?id=26713
+
+        Also change to latin1 encoding of ø to make python happy.
+
+        * Scripts/modules/bugzilla.py:
+        * Scripts/modules/scm.py:
+
+2009-06-25  Eric Seidel  <eric@webkit.org>
+
+        Reviewed by Tor Arne Vestbø.
+
         Add Adam Roben and Tor Arne Vestbø to the reviewers list.
 
         * Scripts/modules/bugzilla.py:
         Add Adam Roben and Tor Arne Vestbø to the reviewers list.
 
         * Scripts/modules/bugzilla.py:
index 573bf38..a788774 100644 (file)
@@ -102,7 +102,7 @@ class Bugzilla:
         "staikos" : "George Staikos",
         "timothy" : "Timothy Hatcher",
         "treat" : "Adam Treat",
         "staikos" : "George Staikos",
         "timothy" : "Timothy Hatcher",
         "treat" : "Adam Treat",
-        "vestbo" : u'Tor Arne Vestb\xc3',
+        "vestbo" : u'Tor Arne Vestb\xf8',
         "xan.lopez" : "Xan Lopez",
         "zecke" : "Holger Freyther",
         "zimmermann" : "Nikolas Zimmermann",
         "xan.lopez" : "Xan Lopez",
         "zecke" : "Holger Freyther",
         "zimmermann" : "Nikolas Zimmermann",
index e226962..60967d2 100644 (file)
@@ -91,7 +91,10 @@ class SCM:
     def apply_patch(self, patch):
         # It's possible that the patch was not made from the root directory.
         # We should detect and handle that case.
     def apply_patch(self, patch):
         # It's possible that the patch was not made from the root directory.
         # We should detect and handle that case.
-        return_code = os.system('curl %s | svn-apply --reviewer "%s"' % (patch['url'], patch['reviewer']))
+        curl_process = subprocess.Popen(['curl', patch['url']], stdout=subprocess.PIPE)
+        patch_apply_process = subprocess.Popen(['svn-apply', '--reviewer', patch['reviewer']], stdin=curl_process.stdout)
+
+        return_code = patch_apply_process.wait()
         if return_code:
             raise ScriptError("Patch " + patch['url'] + " failed to download and apply.")
 
         if return_code:
             raise ScriptError("Patch " + patch['url'] + " failed to download and apply.")