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

Mads Kiilerich mads at kiilerich.com
Mon Sep 29 18:06:24 CDT 2014


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.

/Mads


More information about the Mercurial-devel mailing list