D5981: tests: drop a few unnecessary "(glob)"
mharbison72 (Matt Harbison)
phabricator at mercurial-scm.org
Thu May 9 23:06:06 EDT 2019
mharbison72 added a comment.
In https://phab.mercurial-scm.org/D5981#92414, @martinvonz wrote:
> I had forgotten about this patch and it's not very important anyway. I don't know how I'm supposed to fix it, so I'll just drop. @mharbison72, let me know if there was something else I should have done.
Test stability-wise, we should be OK with or without it. I'm not sure of the intention with the various config options, and Yuya's concern about not changing the output in the thread below.
What's happening now is the commands output paths with '\', and the test harness automatically converts to '/' without needing an explicit `(glob)`. I raised these diffs as an issue at one point because it was outputting with '/', and the test harness complains about those paths having `(glob)` on Windows. It looks like this is basically the same as https://www.mercurial-scm.org/pipermail/mercurial-devel/2019-February/127959.html
To: martinvonz, #hg-reviewers, mharbison72
More information about the Mercurial-devel