Subject: Re: [PATCH v6 00/32] refs backend

From: Ramsay Jones <[email-masked]>
Date: Thu, 25 Feb 2016 00:08:42 +0000

On 24/02/16 22:58, David Turner wrote:
> This version incorporates fixes for function scope from Ramsay Jones.
> 
> It breaks patches down a bit more, to make them easier to review and
> understand. Michael Haggerty had some suggestions here.
> 
> As suggested by Duy Nguyen, it replaces the resolve_ref_unsafe backend
> method with read_raw_ref, moving resolve_ref_unsafe to the common
> code.
> 
> It adds support for running tests under alternate ref backends.
> 
> It fixes two indentation errors, one reported by Eric Wong.
> 
> It updates the documentation to include info about a few more
> limitations on the LMDB backend. It incorporates other LMDB-related
> suggestions from Duy Nguyen, e.g. using MDB_NOSUBDIR.
> 
> There's a fix for non-normal ref renames, also from Duy Nguyen.
> 
> Symbolic ref splitting has a bug fix, suggested by Michael Haggerty.
> 
> There are other, more minor changes suggested by Michael Haggerty and
> Duy Nguyen.
> 
> 
> David Turner (29):
>   refs: move head_ref{,_submodule} to the common code
>   refs: move for_each_*ref* functions into common code
>   files-backend: break out ref reading
>   refs: move resolve_ref_unsafe into common code
>   refs: add method for do_for_each_ref
>   refs: add do_for_each_per_worktree_ref
>   refs: add methods for reflog
>   refs: add method for initial ref transaction commit
>   refs: add method for delete_refs
>   refs: add methods to init refs db
>   refs: add method to rename refs
>   refs: handle non-normal ref renames
>   refs: make lock generic
>   refs: move duplicate check to common code
>   refs: allow log-only updates
>   refs: don't dereference on rename
>   refs: on symref reflog expire, lock symref not referrent
>   refs: resolve symbolic refs first
>   refs: always handle non-normal refs in files backend
>   init: allow alternate ref strorage to be set for new repos
>   refs: check submodules' ref storage config
>   clone: allow ref storage backend to be set for clone
>   svn: learn ref-storage argument
>   refs: register ref storage backends
>   config: read ref storage config on startup
>   refs: break out resolve_ref_unsafe_submodule
>   refs: add LMDB refs storage backend
>   refs: tests for lmdb backend
>   tests: add ref-storage argument
> 
> Ramsay Jones (1):
>   refs: reduce the visibility of do_for_each_ref()

Ah, sorry if it wasn't clear, but there is no need to keep this
as a separate patch - it should simply be squashed into the relevant
patch in your series.

ATB,
Ramsay Jones