narrow: add trailing slash to dir earlier for debug{revlog,index,data}
The treemanifest code internally uses trailing slashes on directories
(except for the root directory, which is an empty string). We should
make sure we pass in directories with trailing slashes when we work
with the treemanifest code. For some reason, I seem to have decided to
be nice to the callers instead in 49c583ca48c4 (treemanifest: add
--dir option to debug{revlog,data,index}, 2015-04-12). Let's fix that
and pay the cost of fixing up the directory name close close to where
we get it from the user.
Differential Revision: https://phab.mercurial-scm.org/D3032
--- a/hgext/narrow/narrowrevlog.py Sat Mar 31 23:49:58 2018 +0530
+++ b/hgext/narrow/narrowrevlog.py Mon Apr 02 23:52:43 2018 -0700
@@ -117,8 +117,6 @@
# at least some push operations. This will be used to wrap/exclude the
# child directories when using treemanifests.
def dirlog(self, d):
- if d and not d.endswith('/'):
- d = d + '/'
if not repo.narrowmatch().visitdir(d[:-1] or '.'):
return excludedmanifestrevlog(d)
result = super(narrowmanifestrevlog, self).dirlog(d)
--- a/mercurial/cmdutil.py Sat Mar 31 23:49:58 2018 +0530
+++ b/mercurial/cmdutil.py Mon Apr 02 23:52:43 2018 -0700
@@ -1058,6 +1058,8 @@
if 'treemanifest' not in repo.requirements:
raise error.Abort(_("--dir can only be used on repos with "
"treemanifest enabled"))
+ if not dir.endswith('/'):
+ dir = dir + '/'
dirlog = repo.manifestlog._revlog.dirlog(dir)
if len(dirlog):
r = dirlog
--- a/mercurial/manifest.py Sat Mar 31 23:49:58 2018 +0530
+++ b/mercurial/manifest.py Mon Apr 02 23:52:43 2018 -0700
@@ -1160,8 +1160,6 @@
if dir:
assert self._treeondisk, 'opts is %r' % opts
- if not dir.endswith('/'):
- dir = dir + '/'
if indexfile is None:
indexfile = '00manifest.i'