On Sun, 2008-06-08 at 16:09 +0000, Dov Feldstern wrote:
> Dov Feldstern <dfeldstern@fastimap.com> added the comment:
> 
> Attached find a patch as Matt suggested.
> 
> I also fixed set_flags itself, although in our current situation this will have
> no effect, since by the time set_flags sees the symlink it will already be a
> link. However, I think that set_flags should do the right thing, without relying
> on it's caller. Alternatively, the entire handling of symlinks in set_flags
> could perhaps be removed?

This is looking better. But I'd really like us to only call the
filesystem check functions once per hg invocation, caching their results
in localrepo.