summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: fcab40a)
raw | patch | inline | side by side (parent: fcab40a)
author | Junio C Hamano <gitster@pobox.com> | |
Thu, 17 Jul 2008 03:25:18 +0000 (20:25 -0700) | ||
committer | Junio C Hamano <gitster@pobox.com> | |
Thu, 17 Jul 2008 06:32:33 +0000 (23:32 -0700) |
This changes the message rerere issues after reusing previous conflict
resolution from "Resolved" to "Staged" when autoupdate option is in
effect.
It is envisioned that in practice, some auto resolitions are trickier and
iffier than others, and we would want to add a feature to mark individual
resolutions as "this is ok to autoupdate" or "do not autoupdate the result
using this resolution even when rerere.autoupdate is in effect" in the
future. When that happens, these messages will make the distinction
clearer.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
resolution from "Resolved" to "Staged" when autoupdate option is in
effect.
It is envisioned that in practice, some auto resolitions are trickier and
iffier than others, and we would want to add a feature to mark individual
resolutions as "this is ok to autoupdate" or "do not autoupdate the result
using this resolution even when rerere.autoupdate is in effect" in the
future. When that happens, these messages will make the distinction
clearer.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
rerere.c | patch | blob | history |
diff --git a/rerere.c b/rerere.c
index 12589215e867bff37d5f2848083e8f138331e8ee..0456aa6376caa1387284aa8fe42fc1ff66629822 100644 (file)
--- a/rerere.c
+++ b/rerere.c
if (has_resolution(name)) {
if (!merge(name, path)) {
- fprintf(stderr, "Resolved '%s' using "
- "previous resolution.\n", path);
if (rerere_autoupdate)
path_list_insert(path, &update);
+ fprintf(stderr,
+ "%s '%s' using previous resolution.\n",
+ rerere_autoupdate
+ ? "Staged" : "Resolved",
+ path);
goto mark_resolved;
}
}