D7649: match: make sure `root` argument is always an absolute path
martinvonz (Martin von Zweigbergk)
phabricator at mercurial-scm.org
Tue Dec 17 15:24:45 UTC 2019
martinvonz added a comment.
In D7649#112837 <https://phab.mercurial-scm.org/D7649#112837>, @mharbison72 wrote:
> Windows has some very strange rules for paths, and I remember fixing a subrepo bug with a path in the form `C:foo`. I tried these, and can't think of anything more we'd need to worry about.
>
> >>> import os
> >>> os.path.isabs("/")
> True
> >>> os.path.isabs("C:/")
> True
> >>> os.path.isabs("C:tests")
> False
>
> (The last one being at the root of the hg repo.)
Thanks for checking. I think that means we're okay with this series now. I won't queue it because I wrote some of it.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST ACTION
https://phab.mercurial-scm.org/D7649/new/
REVISION DETAIL
https://phab.mercurial-scm.org/D7649
To: martinvonz, #hg-reviewers, mharbison72
Cc: mharbison72, mercurial-devel
More information about the Mercurial-devel
mailing list