[PATCH hg-bfiles] Adapt to Mercurial 1.8: util.unlink() is now util.unlinkpath()

Greg Ward greg-hg at gerg.ca
Sun Jan 30 15:18:55 CST 2011


On Sun, Jan 30, 2011 at 11:52 AM, Adrian Buehlmann <adrian at cadifra.com> wrote:
> I'm sorry to have to inflict such a weird API change on extension
> maintainers just to work around another Windows insanity. The new name
> 'unlinkpath' isn't that bad though. It was an idea by Matt.

It's a good change.  Having a wrapper for os.unlink() that *also*
deleted directories was always a bit weird.  Distinguishing the two is
for the best, even if it causes some short-term pain for extension
authors.  You're doing good stuff -- keep it up!

Greg


More information about the Mercurial-devel mailing list