We now have quite a large number of patches before we even get into
the meat of the pluggable refs backend series. So it's worth breaking
those out and getting them in before we get into the main series
(which Michael Haggerty swants to redesign a bit anyway).
This set of patches should be applied on top of
jk/check-repository-format.
Michael Haggerty has reviewed those of my patches which are in here
except maybe:
refs: on symref reflog expire, lock symref not referrent
This was the one from later in the series that was straightforward to
move to before the vtable; the other two were going to be harder to
move and can wait until after the vtable.
I have reviewed all of Michael's patches.
David Turner (5):
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: on symref reflog expire, lock symref not referrent
Michael Haggerty (19):
t1430: test the output and error of some commands more carefully
t1430: clean up broken refs/tags/shadow
t1430: don't rely on symbolic-ref for creating broken symrefs
t1430: test for-each-ref in the presence of badly-named refs
t1430: improve test coverage of deletion of badly-named refs
resolve_missing_loose_ref(): simplify semantics
resolve_ref_unsafe(): use for loop to count up to MAXDEPTH
resolve_ref_unsafe(): ensure flags is always set
resolve_ref_1(): eliminate local variable
resolve_ref_1(): reorder code
resolve_ref_1(): eliminate local variable "bad_name"
read_raw_ref(): manage own scratch space
Inline resolve_ref_1() into resolve_ref_unsafe()
read_raw_ref(): change flags parameter to unsigned int
fsck_head_link(): remove unneeded flag variable
cmd_merge(): remove unneeded flag variable
checkout_paths(): remove unneeded flag variable
check_aliased_update(): check that dst_name is non-NULL
show_head_ref(): check the result of resolve_ref_namespace()
builtin/checkout.c | 3 +-
builtin/fsck.c | 3 +-
builtin/merge.c | 4 +-
builtin/receive-pack.c | 2 +-
http-backend.c | 4 +-
refs.c | 149 ++++++++++++++++++
refs/files-backend.c | 406 ++++++++++++++++--------------------------------
refs/refs-internal.h | 15 ++
t/t1410-reflog.sh | 10 ++
t/t1430-bad-ref-name.sh | 132 ++++++++++++++--
10 files changed, 440 insertions(+), 288 deletions(-)
--
2.4.2.767.g62658d5-twtrsrc
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html