[PATCH 0 of 4] largefiles: make system cache a user cache, and the repo cache a store
greg-hg at gerg.ca
Thu Oct 20 13:49:52 CDT 2011
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
> - 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.
More information about the Mercurial-devel