D5898: debugignore: respect ui.relative-paths

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Sun Feb 17 01:03:09 EST 2019


martinvonz added a comment.


  In https://phab.mercurial-scm.org/D5898#87275, @mharbison72 wrote:
  
  > 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)
  >
  > https://buildbot.mercurial-scm.org/builders/Win7%20x86_64%20hg%20tests/builds/1274/steps/run-tests.py%20%28python%202.7.13%29/logs/stdio
  
  
  It wasn't intentional, and it took me a while to figure out why it changed, but I'm pretty confident it's just changed in the tests, since the test runner sets `ui.slash`. The old code called `m.uipath()` and that called `util.pathto()`. The new code calls `repo.path()`, which calls `dirstate.pathto()`, which also calls `util.pathto()`. However, if `ui.slash` is set, `dirstate.pathto()` also calls `util.pconvert()` to convert it back to a slash-separated path. Please send a patch to update the test case if you have access to a Windows machine.

REPOSITORY
  rHG Mercurial

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

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


More information about the Mercurial-devel mailing list