ikostia requested changes to this revision.
ikostia added a comment.
This revision now requires changes to proceed.


  Please wait until the `util.finddirs()` is tweaked and remove `normpath` call.

INLINE COMMENTS

> yuja wrote in scmutil.py:570
> That's probably okay now.
> 
> I'm not sure if it conflicts with the long paths plan. IIRC, forward
> slash isn't allowed in `\\?\` path.

I think we should adjust `finddirs` to split on `os.path.sep`, not hardcoded 
`/`, then we can get rid of `util.normpath` a couple of lines above.

I am long overdue with re-sending my series, I will add this change there as 
well.

REPOSITORY
  rHG Mercurial

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

To: mbthomas, ryanmce, #hg-reviewers, durham, yuja, ikostia
Cc: ikostia, yuja, durham, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to