# HG changeset patch
# User Matt Harbison <matt_harbison@yahoo.com>
# Date 1527992292 14400
# Node ID 51e420a7a41a9c4b60d7462463c26c8391b06e51
# Parent  36ba5dba372dfe5ac82b6d87a90553ba5dd93586
cmdutil: use internal separators when building the terse list

Status uses internal separators, so this is more correct.  See c974320d20b9 and
362096cfdb1f.  A utility method is still needed, so that the paths starting with
'/' aren't created when `self.path` is empty.

diff -r 36ba5dba372d -r 51e420a7a41a mercurial/cmdutil.py
--- a/mercurial/cmdutil.py	Wed May 30 14:20:09 2018 +0530
+++ b/mercurial/cmdutil.py	Sat Jun 02 22:18:12 2018 -0400
@@ -416,7 +416,7 @@
     Represent a directory in user working copy with information required for
     the purpose of tersing its status.
 
-    path is the path to the directory
+    path is the path to the directory, without a trailing '/'
 
     statuses is a set of statuses of all files in this directory (this includes
     all the files in all the subdirectories too)
@@ -453,7 +453,7 @@
 
             # does the dirnode object for subdir exists
             if subdir not in self.subdirs:
-                subdirpath = os.path.join(self.path, subdir)
+                subdirpath = pathutil.join(self.path, subdir)
                 self.subdirs[subdir] = dirnode(subdirpath)
 
             # try adding the file in subdir
@@ -468,7 +468,7 @@
     def iterfilepaths(self):
         """Yield (status, path) for files directly under this directory."""
         for f, st in self.files:
-            yield st, os.path.join(self.path, f)
+            yield st, pathutil.join(self.path, f)
 
     def tersewalk(self, terseargs):
         """
@@ -482,7 +482,7 @@
 
         1) All the files in the directory (including all the files in its
         subdirectories) share the same status and the user has asked us to terse
-        that status. -> yield (status, dirpath)
+        that status. -> yield (status, dirpath).  dirpath will end in '/'.
 
         2) Otherwise, we do following:
 
@@ -499,7 +499,7 @@
             # Making sure we terse only when the status abbreviation is
             # passed as terse argument
             if onlyst in terseargs:
-                yield onlyst, self.path + pycompat.ossep
+                yield onlyst, self.path + '/'
                 return
 
         # add the files to status list
@@ -551,7 +551,7 @@
     # process each sub-directory and build tersedict
     for subdir in rootobj.subdirs.values():
         for st, f in subdir.tersewalk(terseargs):
-            tersedict[st].append(util.pconvert(f))
+            tersedict[st].append(f)
 
     tersedlist = []
     for st in allst: