[PATCH 0 of 4] largefiles: make system cache a user cache, and the repo cache a store

Benjamin Pollack benjamin at bitquabit.com
Thu Oct 20 13:55:48 CDT 2011

On Oct 20, 2011, at 2:49 PM, Greg Ward wrote:

> On Thu, Oct 20, 2011 at 1:24 PM, Benjamin Pollack
> <benjamin at bitquabit.com> wrote:
>> This patch series does several things:
>>   - In introduces a test authored by Eli Carter that provides a concrete
>>     failure for the current behavior
>>   - It corrects the documentation to clarify what the proper behavior ought to
>>     be
>>   - It renames a large number of functions and methods, plus a config setting,
>>     to actually reflect that desired behavior
>>   - It actually fixes the behavior to match the description, allowing the
>>     failing test to pass
>> The last two, or even three, patches can be folded; they were kept separate for
>> my sanity during development, and for ease of tracking what I actually changed.
>> Likewise, if we're concerned about such a large renaming, the last patch can be
>> easily rewritten to use the old function names.
> Nice! Since patch #1 introduces a failing test, IMHO the commit
> message should say so. Or, better, it should be the last patch, since
> we generally don't like failing tests.

That's fine with me.  Sorry for doing it backwards; our team's practice is to introduce the test first, then the code that allows it to pass, so I just did that without thinking.  I can resubmit in the other order, or it can just be applied differently (it goes cleanly either way).


More information about the Mercurial-devel mailing list