Bug 1331899 - Fix rename handling in wpt sync process, r=Ms2ger draft
authorJames Graham <james@hoppipolla.co.uk>
Tue, 03 Jan 2017 19:06:50 +0000
changeset 462967 0b4edc89061e31c5a532a444e3fb59b0cb2f9ddc
parent 462966 b63b96f6e6bf01f7bf12519accbb5931618f4a2e
child 462968 9cd399f9900ebe2824bb7640f2302a972edf3e0f
push id41928
push userbmo:james@hoppipolla.co.uk
push dateWed, 18 Jan 2017 10:52:19 +0000
reviewersMs2ger
bugs1331899
milestone53.0a1
Bug 1331899 - Fix rename handling in wpt sync process, r=Ms2ger MozReview-Commit-ID: JbBaYGnYKKY
testing/web-platform/update/upstream.py
--- a/testing/web-platform/update/upstream.py
+++ b/testing/web-platform/update/upstream.py
@@ -21,21 +21,22 @@ def rewrite_patch(patch, strip_dir):
     :param patch: the Patch to convert
     :param strip_dir: the path prefix to remove
     """
 
     if not strip_dir.startswith("/"):
         strip_dir = "/%s"% strip_dir
 
     new_diff = []
-    line_starts = ["diff ", "+++ ", "--- ", "rename from ", "rename to "]
+    line_starts = [("diff ", True), ("+++ ", True), ("--- ", True), ("rename from ", False), ("rename to ", False)]
     for line in patch.diff.split("\n"):
-        for start in line_starts:
+        for start, leading_slash in line_starts:
+            strip = strip_dir if leading_slash else strip_dir[1:]
             if line.startswith(start):
-                new_diff.append(line.replace(strip_dir, "").encode("utf8"))
+                new_diff.append(line.replace(strip, "").encode("utf8"))
                 break
         else:
             new_diff.append(line)
 
     new_diff = "\n".join(new_diff)
 
     assert new_diff != patch