Why is not platform specific 'realpath' used instead of 'os.path.realpath' ?

Pierre-Yves David pierre-yves.david at ens-lyon.org
Thu Aug 9 05:32:40 CDT 2012


On 9 août 2012, at 12:30, Mads Kiilerich wrote:
> Some digging shows that it was used in http://selenic.com/repo/hg/rev/26fa0e31011d but backed out again in http://selenic.com/repo/hg/rev/f6683a7011e9 .
> 
> It was discussed around http://www.selenic.com/pipermail/mercurial-devel/2011-October/ . It was fixed http://selenic.com/repo/hg/rev/ab600a25dfc0 and used http://selenic.com/repo/hg/rev/42630f54e513 and used everywhere http://selenic.com/repo/hg/rev/dbdb777502dc ... and all uses backed out again in http://selenic.com/repo/hg/rev/c519cd8f0169 .
> 
> Here be dragons!
> 
> But yes, all uses of os.path should IMO go away and be replaced by the VFS (or other means).

This sound like something that need a check-code.py rule.

-- 
Pierre-Yves


More information about the Mercurial-devel mailing list