Re: [PATCH v6 00/32] refs backend
- From
Ramsay Jones <ramsay@ramsayjones.plus.com>
- Date
- Feb 25, 2016, 00:08 UTC
- Message-ID
- <56CE460A.4020505@ramsayjones.plus.com>
- In-Reply-To
- <1456354744-8022-1-git-send-email-dturner@twopensource.com>
On 24/02/16 22:58, David Turner wrote:
Show 58 quoted lines
> 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