[PATCH 1 of 8] largefiles: replace "_isXXXXing" attributes with "_lfautocommit"

Matt Mackall mpm at selenic.com
Mon Sep 29 18:15:17 CDT 2014


On Tue, 2014-09-30 at 01:06 +0200, Mads Kiilerich wrote:
> On 09/30/2014 12:51 AM, Matt Mackall wrote:
> > On Thu, 2014-09-25 at 00:43 +0200, Mads Kiilerich wrote:
> >> I think they are valid. There is no perfect solution ... and probably
> >> not even a good one. But I think we should reconsider whether the
> >> current approach of violating all layers and add hacks everywhere is the
> >> best solution.
> > I think if we know how to get largefiles more correct (for instance, by
> > applying this patch series), then we should do that. While this might
> > increase the complexity, it also puts us in a better place for measuring
> > regressions when refactoring.. and also delivers immediate benefits to
> > users.
> 
> Fine with me. I would love to get the benefits too.

I guess we're waiting for a resend in any case.

-- 
Mathematics is the supreme nostalgia of our time.




More information about the Mercurial-devel mailing list