# HG changeset patch # User Augie Fackler # Date 1432665048 14400 # Node ID 46f2df2f0680ee6dcee9d1a3a167b9e30be35628 # Parent 7072b91ccd20da95798381fb2e53bb7d41b041c0 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. diff -r 7072b91ccd20 -r 46f2df2f0680 mercurial/pathutil.py --- 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)