D5981: tests: drop a few unnecessary "(glob)"
mharbison72 (Matt Harbison)
phabricator at mercurial-scm.org
Mon Feb 18 12:32:12 EST 2019
mharbison72 added a comment.
The tests run clean with this.
But I'm a bit confused. The test harness has been doing '\' -> '/' conversion without (glob) now for a little over a year, and it complains if there's a trailing (glob) and no '\' -> '/' conversion. That's not been happening here, and made me suspicious. These globs predate that functionality slightly, so I'm not sure the meaning of the referenced commit (which seems to say nothing will change because ui.slash is set).
To be clear, these paths are still being printed with '\', and the test harness is accommodating that. But I'm not sure if that's what you intended.
To: martinvonz, #hg-reviewers, mharbison72
More information about the Mercurial-devel