This is the sixth iteration of the long-awaited re-roll of the attempt to
avoid spawning merge-recursive from the builtin am and use merge_recursive()
directly instead.
The *real* reason for the reroll is that I need a libified recursive
merge to accelerate the interactive rebase by teaching the sequencer to
do rebase -i's grunt work. Coming with a very nice 3x-5x speedup of
`rebase -i`.
In this endeavor, we need to be extra careful to retain backwards
compatibility. The test script t6022-merge-rename.sh, for example, verifies
that `git pull` exits with status 128 in case of a fatal error. To that end,
we need to make sure that fatal errors are handled by existing (builtin)
users via exit(128) (or die(), which calls exit(128) at the end). New users
(such as a builtin helper doing rebase -i's grunt work) may want to print
some helpful advice what happened and how to get out of this mess before
erroring out.
The changes relative to the fifth iteration of this patch series:
- the commit message that talked about swallowing messages was improved
- the va_start()/va_end() statements were moved closer to the vararg usage
- the `buffer_output` field is no longer a bit field because it is now
logically more like the verbosity level, which we traditionally keep
as a full `int`
- the flush_output() call is no longer moved, but instead added to the
code path when we return early
- the commit message saying that we speculated about stdout's buffer
size now instead states that we made an assumption
This patch series touches rather important code. I appreciate thorough
reviews with a focus on the critical parts of the code, those that could
result in regressions.
Johannes Schindelin (16):
t5520: verify that `pull --rebase` shows the helpful advice when
failing
Report bugs consistently
Avoid translating bug messages
merge-recursive: clarify code in was_tracked()
Prepare the builtins for a libified merge_recursive()
merge_recursive: abort properly upon errors
merge-recursive: avoid returning a wholesale struct
merge-recursive: allow write_tree_from_memory() to error out
merge-recursive: handle return values indicating errors
merge-recursive: switch to returning errors instead of dying
am -3: use merge_recursive() directly again
merge-recursive: flush output buffer before printing error messages
merge-recursive: write the commit title in one go
merge-recursive: offer an option to retain the output in 'obuf'
Ensure that the output buffer is released after calling merge_trees()
merge-recursive: flush output buffer even when erroring out
builtin/am.c | 62 ++----
builtin/checkout.c | 5 +-
builtin/ls-files.c | 3 +-
builtin/merge.c | 2 +
builtin/update-index.c | 2 +-
grep.c | 8 +-
imap-send.c | 2 +-
merge-recursive.c | 578 +++++++++++++++++++++++++++++--------------------
merge-recursive.h | 2 +-
sequencer.c | 5 +
sha1_file.c | 4 +-
t/t5520-pull.sh | 32 +++
trailer.c | 2 +-
transport.c | 2 +-
wt-status.c | 4 +-
15 files changed, 419 insertions(+), 294 deletions(-)
Published-As: https://github.com/dscho/git/releases/tag/am-3-merge-recursive-direct-v6
Interdiff vs v5:
diff --git a/merge-recursive.c b/merge-recursive.c
index 66e93e0..c9e4dbc 100644
--- a/merge-recursive.c
+++ b/merge-recursive.c
@@ -35,21 +35,21 @@ static int err(struct merge_options *o, const char *err, ...)
{
va_list params;
- va_start(params, err);
if (o->buffer_output < 2)
flush_output(o);
else {
strbuf_complete(&o->obuf, '\n');
strbuf_addstr(&o->obuf, "error: ");
}
+ va_start(params, err);
strbuf_vaddf(&o->obuf, err, params);
+ va_end(params);
if (o->buffer_output > 1)
strbuf_addch(&o->obuf, '\n');
else {
error("%s", o->obuf.buf);
strbuf_reset(&o->obuf);
}
- va_end(params);
return -1;
}
@@ -2069,16 +2069,18 @@ int merge_recursive(struct merge_options *o,
o->ancestor = "merged common ancestors";
clean = merge_trees(o, h1->tree, h2->tree, merged_common_ancestors->tree,
&mrtree);
- flush_output(o);
- if (clean < 0)
+ if (clean < 0) {
+ flush_output(o);
return clean;
+ }
if (o->call_depth) {
*result = make_virtual_commit(mrtree, "merged tree");
commit_list_insert(h1, &(*result)->parents);
commit_list_insert(h2, &(*result)->parents->next);
}
- if (o->buffer_output < 2)
+ flush_output(o);
+ if (!o->call_depth && o->buffer_output < 2)
strbuf_release(&o->obuf);
if (show(o, 2))
diff_warn_rename_limit("merge.renamelimit",
diff --git a/merge-recursive.h b/merge-recursive.h
index 340704c..735343b 100644
--- a/merge-recursive.h
+++ b/merge-recursive.h
@@ -13,7 +13,7 @@ struct merge_options {
MERGE_RECURSIVE_THEIRS
} recursive_variant;
const char *subtree_shift;
- unsigned buffer_output : 2; /* 1: output at end, 2: keep buffered */
+ unsigned buffer_output; /* 1: output at end, 2: keep buffered */
unsigned renormalize : 1;
long xdl_opts;
int verbosity;
--
2.9.0.281.g286a8d9
base-commit: f8f7adce9fc50a11a764d57815602dcb818d1816
--
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