+ $ rm -r narrow-via-localpeer
+ rm: narrow-via-localpeer: $ENOENT$
+ [1]

+ for result in results:
+ r = result.result()

No idea which is more correct, but drawdag.py is generally paired with
$TESTDIR. Queued, thanks.

Looks good to me. One nit.

Maybe need b'repo'.

Added workaround for test-check-help. as follows:

Queued the series, thanks.

self.automatestdio = True

• self.ui.debug("mtn automate version %s - using automate stdio\n" % + self.ui.debug("mtn automate version %f - using automate stdio\n" % version)

Looks good. Queued the series, thanks.

+def filterknowncreateopts(ui, createopts):
+ """Filters a dict of repo creation options against options that are known.

> Any reason not to use `self._loadlazy(kslash)`?

Hmm, I feel like there was a previous version that iterated differently and meant I'd need a copy of the list to do this correctly.. in this version, no I don't think there's a reason to avoid it besides a potential minor speed difference; should I send a patch for that?

+ def _loadchildrensetlazy(self, visit):
+ if not visit:
+ return None
+ if visit == 'all' or visit == 'this':
+ self._loadalllazy()
+ return None
+
+ todel = []
+ for k in visit:
+ kslash = k + '/'
+ ld = self._lazydirs.get(kslash)
+ if ld:
+ path, node, readsubtree = ld
+ self._dirs[kslash] = readsubtree(path, node)
+ todel.append(kslash)
+ for kslash in todel:
+ del self._lazydirs[kslash]

Queued, thanks.

Perhaps iter() needs to be extracted to a free function to avoid
reference cycle.

Queued the series, thanks.

+def purge(repo, dirs=None, ignored=False, removeemptydirs=True,
+ removefiles=True, abortonerror=False, include=None, exclude=None,
+ noop=False):

Queued for stable, thanks.

Queued, thanks.

Queued up to this patch, thanks. The decoder looks promising.

Queued, thanks.

def toggletrace(cmd):

• quoted = shellquote(cmd.strip()).replace(b'\\', b'\\\\') + cmd = cmd.decode('ascii') + quoted = shellquote(cmd.strip()).encode('ascii') + quoted = quoted.replace(b'\\', b'\\\\')

@@ -3034,7 +3035,19 @@

prefetch(repo, [ctx.rev()],
         matchfiles(repo,
                    [f for sublist in oplist for f in sublist]))

• _performrevert(repo, parents, ctx, actions, interactive, tobackup) + # give status messages for actions to be performed + acts = "add drop undelete forget remove revert".split() + if interactive: + # exclude ("forget", "remove", "revert") as status msg for these + # actions will be handled in interactive session. + acts = acts[:3] + for act in acts: + for f in actions[act][0]: + rel, exact = names[f] + if ui.verbose or not exact: + ui.status(actions[act][1] % rel) + _performrevert(repo, parents, ctx, names, actions, interactive, + tobackup)

> + if ui.verbose or not exact:
> + if not isinstance(msg, bytes):
> + msg = msg(abs)

Does this conditional doing something useful because I don't see any change in the tests if I remove line 3023,3024?

+ elif boolctrace is False:
+ # stringutil.parsebool() returns None when it is unable to parse the
+ # value, so we should rely on making sure copytracing is on such cases