util: don't encode ':' in url paths
authorMads Kiilerich <mads@kiilerich.com>
Mon, 07 Nov 2011 03:25:10 +0100
changeset 15452 de7e2fba4326
parent 15451 23144f06919c
child 15453 cff509500a24
child 15467 c26db53dc024
util: don't encode ':' in url paths ':' has no special meaning in paths, so there is no need for encoding it. Not encoding ':' makes it easier to test on windows.
mercurial/util.py
tests/test-url.py
--- a/mercurial/util.py	Mon Nov 07 03:25:10 2011 +0100
+++ b/mercurial/util.py	Mon Nov 07 03:25:10 2011 +0100
@@ -1496,7 +1496,7 @@
     """
 
     _safechars = "!~*'()+"
-    _safepchars = "/!~*'()+"
+    _safepchars = "/!~*'()+:"
     _matchscheme = re.compile(r'^[a-zA-Z0-9+.\-]+:').match
 
     def __init__(self, path, parsequery=True, parsefragment=True):
@@ -1608,8 +1608,8 @@
 
         Examples:
 
-        >>> str(url('http://user:pw@host:80/?foo#bar'))
-        'http://user:pw@host:80/?foo#bar'
+        >>> str(url('http://user:pw@host:80/c:/bob?fo:oo#ba:ar'))
+        'http://user:pw@host:80/c:/bob?fo:oo#ba:ar'
         >>> str(url('http://user:pw@host:80/?foo=bar&baz=42'))
         'http://user:pw@host:80/?foo=bar&baz=42'
         >>> str(url('http://user:pw@host:80/?foo=bar%3dbaz'))
--- a/tests/test-url.py	Mon Nov 07 03:25:10 2011 +0100
+++ b/tests/test-url.py	Mon Nov 07 03:25:10 2011 +0100
@@ -219,7 +219,7 @@
     >>> u
     <url scheme: 'file', path: 'f:oo/bar/baz'>
     >>> str(u)
-    'file:f%3Aoo/bar/baz'
+    'file:f:oo/bar/baz'
     >>> u.localpath()
     'f:oo/bar/baz'