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.

  rHG Mercurial


To: martinvonz, #hg-reviewers, mharbison72
Cc: mercurial-devel

More information about the Mercurial-devel mailing list