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.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D5981

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


More information about the Mercurial-devel mailing list