changeset 25285:46f2df2f0680

pathutil: restate dirname and join as forwards to posixpath I've done this as its own step so that it's easy to see that the posixpath implementations pass the doctests in this package. In a future patch I'll just make these pure forwards of the methods so that things using pathutil can be oblivious to the posix nature of these functions.
author Augie Fackler <augie@google.com>
date Tue, 26 May 2015 14:30:48 -0400
parents 7072b91ccd20
children 127a11f705d9
files mercurial/pathutil.py
diffstat 1 files changed, 4 insertions(+), 19 deletions(-) [+]
line wrap: on
line diff
--- a/mercurial/pathutil.py	Wed May 20 14:54:09 2015 -0700
+++ b/mercurial/pathutil.py	Tue May 26 14:30:48 2015 -0400
@@ -1,4 +1,4 @@
-import os, errno, stat
+import os, errno, stat, posixpath
 
 import encoding
 import util
@@ -188,7 +188,7 @@
     else:
         return path
 
-def join(path, *paths):
+def join(*args):
     '''Join two or more pathname components, inserting '/' as needed.
 
     Based on the posix os.path.join() implementation.
@@ -214,17 +214,7 @@
     >>> join ('foo', '', '', 'bar')
     'foo/bar'
     '''
-    sep = '/'
-    if not paths:
-        path[:0] + sep #23780: Ensure compatible data type even if p is null.
-    for piece in paths:
-        if piece.startswith(sep):
-            path = piece
-        elif not path or path.endswith(sep):
-            path += piece
-        else:
-            path += sep + piece
-    return path
+    return posixpath.join(*args)
 
 def dirname(path):
     '''returns the directory portion of the given path
@@ -246,9 +236,4 @@
     >>> dirname('/foo//bar')
     '/foo'
     '''
-    sep = '/'
-    i = path.rfind(sep) + 1
-    dirname = path[:i]
-    if dirname and dirname != sep * len(dirname):
-        dirname = dirname.rstrip(sep)
-    return dirname
+    return posixpath.dirname(path)