Skip to content

Commit

Permalink
update-index: do not read HEAD and MERGE_HEAD unconditionally
Browse files Browse the repository at this point in the history
When "update-index --unresolve $path" cannot find the resolve-undo
record for the path the user requested to unresolve, it stuffs the
blobs from HEAD and MERGE_HEAD to stage #2 and stage #3 as a
fallback.  For this reason, the operation does not even start unless
both "HEAD" and "MERGE_HEAD" exist.

This is suboptimal in a few ways:

 * It does not recreate stage #1.  Even though it is a correct
   design decision not to do so (because it is impossible to
   recreate in general cases, without knowing how we got there,
   including what merge strategy was used), it is much less useful
   not to have that information in the index.

 * It limits the "unresolve" operation only during a conflicted "git
   merge" and nothing else.  Other operations like "rebase",
   "cherry-pick", and "switch -m" may result in conflicts, and the
   user may want to unresolve the conflict that they incorrectly
   resolved in order to redo the resolution, but the fallback would
   not kick in.

 * Most importantly, the entire "unresolve" operation is disabled
   after a conflicted merge is committed and MERGE_HEAD is removed,
   even though the index has perfectly usable resolve-undo records.

By lazily reading the HEAD and MERGE_HEAD only when we need to go to
the fallback codepath, we will allow cases where resolve-undo
records are available (which is 100% of the time, unless the user is
reading from an index file created by Git more than 10 years ago) to
proceed even after a conflicted merge was committed, during other
mergy operations that do not use MERGE_HEAD, or after the result of
such mergy operations has been committed.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
gitster committed Jul 31, 2023
1 parent fb7d80e commit 91e0705
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 18 deletions.
46 changes: 28 additions & 18 deletions builtin/update-index.c
Original file line number Diff line number Diff line change
Expand Up @@ -639,6 +639,21 @@ static struct cache_entry *read_one_ent(const char *which,
return ce;
}

static int read_head_pointers(void)
{
static int result = -2; /* unknown yet */

if (result == -2) {
result = -1;
if (read_ref("HEAD", &head_oid))
return error("No HEAD -- no initial commit yet?");
if (read_ref("MERGE_HEAD", &merge_head_oid))
return error("Not in the middle of a merge");
result = 0;
}
return result;
}

static int unresolve_one(const char *path)
{
int namelen = strlen(path);
Expand Down Expand Up @@ -677,10 +692,20 @@ static int unresolve_one(const char *path)
}
}

/* Grab blobs from given path from HEAD and MERGE_HEAD,
* stuff HEAD version in stage #2,
* stuff MERGE_HEAD version in stage #3.
/*
* We are not using resolve-undo information but just
* populating the stages #2 and #3 from HEAD and MERGE_HEAD.
*
* This is a flawed replacement of true "unresolve", as we do
* not have a way to recreate the stage #1 for the common
* ancestor (which may not be a unique merge-base between the
* two).
*/
if (read_head_pointers()) {
ret = -1;
goto free_return;
}

ce_2 = read_one_ent("our", &head_oid, path, namelen, 2);
ce_3 = read_one_ent("their", &merge_head_oid, path, namelen, 3);

Expand Down Expand Up @@ -711,27 +736,12 @@ static int unresolve_one(const char *path)
return ret;
}

static void read_head_pointers(void)
{
if (read_ref("HEAD", &head_oid))
die("No HEAD -- no initial commit yet?");
if (read_ref("MERGE_HEAD", &merge_head_oid)) {
fprintf(stderr, "Not in the middle of a merge.\n");
exit(0);
}
}

static int do_unresolve(int ac, const char **av,
const char *prefix, int prefix_length)
{
int i;
int err = 0;

/* Read HEAD and MERGE_HEAD; if MERGE_HEAD does not exist, we
* are not doing a merge, so exit with success status.
*/
read_head_pointers();

for (i = 1; i < ac; i++) {
const char *arg = av[i];
char *p = prefix_path(prefix, prefix_length, arg);
Expand Down
8 changes: 8 additions & 0 deletions t/t2030-unresolve-info.sh
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,14 @@ test_expect_success 'unmerge with plumbing' '
test_line_count = 3 actual
'

test_expect_success 'unmerge can be done even after committing' '
prime_resolve_undo &&
git commit -m "record to nuke MERGE_HEAD" &&
git update-index --unresolve fi/le &&
git ls-files -u >actual &&
test_line_count = 3 actual
'

test_expect_success 'rerere and rerere forget' '
mkdir .git/rr-cache &&
prime_resolve_undo &&
Expand Down

0 comments on commit 91e0705

Please sign in to comment.