From: Ramsay Jones <ramsay@ramsayjones•plus.com>
To: David Turner <dturner@twopensource•com>,
git@vger•kernel.org, mhagger@alum•mit.edu, pclouds@gmail•com
Subject: Re: [PATCH v6 00/32] refs backend
Date: Thu, 25 Feb 2016 00:08:42 +0000 [thread overview]
Message-ID: <56CE460A.4020505@ramsayjones.plus.com> (raw)
In-Reply-To: <1456354744-8022-1-git-send-email-dturner@twopensource.com>
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
next prev parent reply other threads:[~2016-02-25 0:08 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-24 22:58 [PATCH v6 00/32] refs backend David Turner
2016-02-24 22:58 ` [PATCH v6 01/32] refs: move head_ref{,_submodule} to the common code David Turner
2016-02-24 22:58 ` [PATCH v6 02/32] refs: move for_each_*ref* functions into " David Turner
2016-02-24 22:58 ` [PATCH v6 03/32] files-backend: break out ref reading David Turner
2016-02-25 0:21 ` Junio C Hamano
2016-02-25 0:51 ` Junio C Hamano
2016-02-25 20:11 ` David Turner
2016-02-25 20:27 ` Junio C Hamano
2016-02-24 22:58 ` [PATCH v6 04/32] refs: move resolve_ref_unsafe into common code David Turner
2016-02-25 0:22 ` Junio C Hamano
2016-02-24 22:58 ` [PATCH v6 05/32] refs: add a backend method structure with transaction functions David Turner
2016-02-27 4:06 ` Jeff King
2016-02-29 20:43 ` David Turner
2016-02-24 22:58 ` [PATCH v6 06/32] refs: add methods for misc ref operations David Turner
2016-02-24 22:58 ` [PATCH v6 07/32] refs: add method for do_for_each_ref David Turner
2016-02-24 22:58 ` [PATCH v6 08/32] refs: reduce the visibility of do_for_each_ref() David Turner
2016-02-24 22:58 ` [PATCH v6 09/32] refs: add do_for_each_per_worktree_ref David Turner
2016-02-24 22:58 ` [PATCH v6 10/32] refs: add methods for reflog David Turner
2016-02-24 22:58 ` [PATCH v6 11/32] refs: add method for initial ref transaction commit David Turner
2016-02-24 22:58 ` [PATCH v6 12/32] refs: add method for delete_refs David Turner
2016-02-24 22:58 ` [PATCH v6 13/32] refs: add methods to init refs db David Turner
2016-02-24 22:58 ` [PATCH v6 14/32] refs: add method to rename refs David Turner
2016-02-24 22:58 ` [PATCH v6 15/32] refs: handle non-normal ref renames David Turner
2016-02-24 22:58 ` [PATCH v6 16/32] refs: make lock generic David Turner
2016-02-24 22:58 ` [PATCH v6 17/32] refs: move duplicate check to common code David Turner
2016-02-24 22:58 ` [PATCH v6 18/32] refs: allow log-only updates David Turner
2016-02-24 22:58 ` [PATCH v6 19/32] refs: don't dereference on rename David Turner
2016-02-24 22:58 ` [PATCH v6 20/32] refs: on symref reflog expire, lock symref not referrent David Turner
2016-02-24 22:58 ` [PATCH v6 21/32] refs: resolve symbolic refs first David Turner
2016-02-24 22:58 ` [PATCH v6 22/32] refs: always handle non-normal refs in files backend David Turner
2016-02-24 22:58 ` [PATCH v6 23/32] init: allow alternate ref strorage to be set for new repos David Turner
2016-02-24 22:58 ` [PATCH v6 24/32] refs: check submodules' ref storage config David Turner
2016-02-24 22:58 ` [PATCH v6 25/32] clone: allow ref storage backend to be set for clone David Turner
2016-02-24 22:58 ` [PATCH v6 26/32] svn: learn ref-storage argument David Turner
2016-02-24 22:58 ` [PATCH v6 27/32] refs: register ref storage backends David Turner
2016-02-24 22:59 ` [PATCH v6 28/32] config: read ref storage config on startup David Turner
2016-02-27 3:56 ` Jeff King
2016-02-27 4:14 ` Jeff King
2016-02-29 21:57 ` David Turner
2016-02-29 22:13 ` Jeff King
2016-02-29 21:57 ` David Turner
2016-02-29 22:10 ` Jeff King
2016-02-29 22:21 ` Jeff King
2016-02-29 22:59 ` David Turner
2016-02-24 22:59 ` [PATCH v6 29/32] refs: break out resolve_ref_unsafe_submodule David Turner
2016-02-24 22:59 ` [PATCH v6 30/32] refs: add LMDB refs storage backend David Turner
2016-02-24 22:59 ` [PATCH v6 31/32] refs: tests for lmdb backend David Turner
2016-02-24 22:59 ` [PATCH v6 32/32] tests: add ref-storage argument David Turner
2016-02-29 22:52 ` SZEDER Gábor
2016-02-29 23:50 ` David Turner
2016-02-25 0:08 ` Ramsay Jones [this message]
2016-02-25 19:32 ` [PATCH v6 00/32] refs backend David Turner
2016-02-25 12:57 ` Duy Nguyen
2016-02-25 19:27 ` David Turner
2016-02-25 19:31 ` Stefan Beller
2016-02-25 20:31 ` Duy Nguyen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56CE460A.4020505@ramsayjones.plus.com \
--to=ramsay@ramsayjones$(echo .)plus.com \
--cc=dturner@twopensource$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=mhagger@alum$(echo .)mit.edu \
--cc=pclouds@gmail$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox