I noticed another bug and sent D5978. Maybe your test failure is because you're using the eol extension? I don't know what else would cause the \r in contrib.perf. I have no idea how that's related to this patch, though.
The \r is how output normally is on Windows. The test harness accounts for this when matching lines, but displays the actual output (with \r) when there are differences. The eol extension isn’t configured on this machine, but the custom HGRCPATH content from the test harness would override that anyway.
For some reason, this breaks test-contrib-perf.t:
I also tried setting True and False in ui.relative-paths, to no effect. Any ideas?
This seems to have converted a test from dos to unix style. I'm not sure if that was intentional, or if this feature is still a WIP. (Sorry, I've been busy lately, and haven't had time to follow this closely)
Fri, Feb 15
Thu, Feb 14
Looks like there's plenty of space there to show the full username. I suspect it was the way it was because it came from FB where just the username is enough to identify people.
Wed, Feb 13
Ironic given the subject line that I missed this hunk:diff --git a/mercurial/commands.py b/mercurial/commands.py --- a/mercurial/commands.py +++ b/mercurial/commands.py @@ -256,7 +256,7 @@ def addremove(ui, repo, *pats, **opts): opts['similarity'] = '100' matcher = scmutil.match(repo[None], pats, opts) relative = scmutil.anypats(pats, opts) - uipathfn = scmutil.getuipathfn(repo, forcerelativevalue=relative) + uipathfn = scmutil.getuipathfn(repo, legacyrelativevalue=relative) return scmutil.addremove(repo, matcher, "", uipathfn, opts) @command('annotate|blame',
I'll fold that in in flight since the commit is not yet public.
Ironic given the subject line that I missed this hunk:
Tue, Feb 12
So, I haven't been here in a while. Do we still do V2s for this kind of thing?
fix typo in synpsis
Mon, Feb 11
`getuipathfn()` uses `repo.pathto()` when a relative path was requested (including by setting `legacyrelativevalue=True` or `forcerelativevalue=True`), so I think there shouldn't be much change in behavior with that commit.
But the default of hg status is relative=False, so the status output
would have significant change on Windows.
> It might > break some Windows tests, but it should already be broken since https://phab.mercurial-scm.org/rHGa997163e7fae2fe933f8d0c6d1013205befd1ee4. > Some (or all?) `uipathfn()`s have to respect `ui.slash` config on Windows. Did that commit change any behavior on Windows?
Probably. pathto(f, cwd='') would return backslash path on Windows.