mirror of
https://github.com/git/git.git
synced 2024-10-28 04:49:43 +01:00
992f7a4fdb
For each linked worktree, Git maintains two pointers: (1)
<repo>/worktrees/<id>/gitdir which points at the linked worktree, and
(2) <worktree>/.git which points back at <repo>/worktrees/<id>. Both
pointers are absolute pathnames.
Aside from manually manipulating those raw files, it is possible to
easily "break" one or both pointers by ignoring the "git worktree move"
command and instead manually moving a linked worktree, moving the
repository, or moving both. The "git worktree repair" command was
invented to handle this case by restoring these pointers to sane values.
For the "repair" command, the "git worktree" manual page states:
Repair worktree administrative files, if possible, if they have
become corrupted or outdated due to external factors.
The "if possible" clause was chosen deliberately to convey that the
existing implementation may not be able to fix every possible breakage,
and to imply that improvements may be made to handle other types of
breakage.
A recent problem report[*] illustrates a case in which "git worktree
repair" not only fails to fix breakage, but actually causes breakage.
Specifically, if a repository / main-worktree and linked worktrees are
*copied* as a unit (rather than *moved*), then "git worktree repair" run
in the copy leaves the copy untouched but botches the pointers in the
original repository and the original worktrees.
For instance, given this directory structure:
orig/
main/ (main-worktree)
linked/ (linked worktree)
if "orig" is copied (not moved) to "dup", then immediately after the
manual copy operation:
* orig/main/.git/worktrees/linked/gitdir points at orig/linked/.git
* orig/linked/.git points at orig/main/.git/worktrees/linked
* dup/main/.git/worktrees/linked/gitdir points at orig/linked/.git
* dup/linked/.git points at orig/main/.git/worktrees/linked
So, dup/main thinks its linked worktree is orig/linked, and worktree
dup/linked thinks its repository / main-worktree is orig/main.
"git worktree repair" is reasonably simple-minded; it wants to trust
valid-looking pointers, hence doesn't try to second-guess them. In this
case, when validating dup/linked/.git, it finds a legitimate repository
pointer, orig/main/.git/worktrees/linked, thus trusts that is correct,
but does notice that gitdir in that directory doesn't point at
dup/linked/.git, so it (incorrectly) _fixes_
orig/main/.git/worktrees/linked/gitdir to point at dup/linked/.git.
Similarly, when validating dup/main/.git/worktrees/linked/gitdir, it
finds a legitimate worktree pointer, orig/linked/.git, but notices that
its .git file doesn't point back at dup/main, thus (incorrectly) _fixes_
orig/linked/.git to point at dup/main/.git/worktrees/linked. Hence, it
has modified and broken the linkage between orig/main and orig/linked
rather than fixing dup/main and dup/linked as expected.
Fix this problem by also checking if a plausible .git/worktrees/<id>
exists in the *current* repository -- not just in the repository pointed
at by the worktree's .git file -- and comparing whether they are the
same. If not, then it is likely because the repository / main-worktree
and linked worktrees were copied, so prefer the discovered plausible
pointer rather than the one from the existing .git file.
[*]: https://lore.kernel.org/git/E1sr5iF-0007zV-2k@binarylane-bailey.stuart.id.au/
Reported-by: Russell Stuart <russell+git.vger.kernel.org@stuart.id.au>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
219 lines
7.2 KiB
Bash
Executable file
219 lines
7.2 KiB
Bash
Executable file
#!/bin/sh
|
|
|
|
test_description='test git worktree repair'
|
|
|
|
TEST_PASSES_SANITIZE_LEAK=true
|
|
. ./test-lib.sh
|
|
|
|
test_expect_success setup '
|
|
test_commit init
|
|
'
|
|
|
|
test_expect_success 'skip missing worktree' '
|
|
test_when_finished "git worktree prune" &&
|
|
git worktree add --detach missing &&
|
|
rm -rf missing &&
|
|
git worktree repair >out 2>err &&
|
|
test_must_be_empty out &&
|
|
test_must_be_empty err
|
|
'
|
|
|
|
test_expect_success 'worktree path not directory' '
|
|
test_when_finished "git worktree prune" &&
|
|
git worktree add --detach notdir &&
|
|
rm -rf notdir &&
|
|
>notdir &&
|
|
test_must_fail git worktree repair >out 2>err &&
|
|
test_must_be_empty out &&
|
|
test_grep "not a directory" err
|
|
'
|
|
|
|
test_expect_success "don't clobber .git repo" '
|
|
test_when_finished "rm -rf repo && git worktree prune" &&
|
|
git worktree add --detach repo &&
|
|
rm -rf repo &&
|
|
test_create_repo repo &&
|
|
test_must_fail git worktree repair >out 2>err &&
|
|
test_must_be_empty out &&
|
|
test_grep ".git is not a file" err
|
|
'
|
|
|
|
test_corrupt_gitfile () {
|
|
butcher=$1 &&
|
|
problem=$2 &&
|
|
repairdir=${3:-.} &&
|
|
test_when_finished 'rm -rf corrupt && git worktree prune' &&
|
|
git worktree add --detach corrupt &&
|
|
git -C corrupt rev-parse --absolute-git-dir >expect &&
|
|
eval "$butcher" &&
|
|
git -C "$repairdir" worktree repair 2>err &&
|
|
test_grep "$problem" err &&
|
|
git -C corrupt rev-parse --absolute-git-dir >actual &&
|
|
test_cmp expect actual
|
|
}
|
|
|
|
test_expect_success 'repair missing .git file' '
|
|
test_corrupt_gitfile "rm -f corrupt/.git" ".git file broken"
|
|
'
|
|
|
|
test_expect_success 'repair bogus .git file' '
|
|
test_corrupt_gitfile "echo \"gitdir: /nowhere\" >corrupt/.git" \
|
|
".git file broken"
|
|
'
|
|
|
|
test_expect_success 'repair incorrect .git file' '
|
|
test_when_finished "rm -rf other && git worktree prune" &&
|
|
test_create_repo other &&
|
|
other=$(git -C other rev-parse --absolute-git-dir) &&
|
|
test_corrupt_gitfile "echo \"gitdir: $other\" >corrupt/.git" \
|
|
".git file incorrect"
|
|
'
|
|
|
|
test_expect_success 'repair .git file from main/.git' '
|
|
test_corrupt_gitfile "rm -f corrupt/.git" ".git file broken" .git
|
|
'
|
|
|
|
test_expect_success 'repair .git file from linked worktree' '
|
|
test_when_finished "rm -rf other && git worktree prune" &&
|
|
git worktree add --detach other &&
|
|
test_corrupt_gitfile "rm -f corrupt/.git" ".git file broken" other
|
|
'
|
|
|
|
test_expect_success 'repair .git file from bare.git' '
|
|
test_when_finished "rm -rf bare.git corrupt && git worktree prune" &&
|
|
git clone --bare . bare.git &&
|
|
git -C bare.git worktree add --detach ../corrupt &&
|
|
git -C corrupt rev-parse --absolute-git-dir >expect &&
|
|
rm -f corrupt/.git &&
|
|
git -C bare.git worktree repair &&
|
|
git -C corrupt rev-parse --absolute-git-dir >actual &&
|
|
test_cmp expect actual
|
|
'
|
|
|
|
test_expect_success 'invalid worktree path' '
|
|
test_must_fail git worktree repair /notvalid >out 2>err &&
|
|
test_must_be_empty out &&
|
|
test_grep "not a valid path" err
|
|
'
|
|
|
|
test_expect_success 'repo not found; .git not file' '
|
|
test_when_finished "rm -rf not-a-worktree" &&
|
|
test_create_repo not-a-worktree &&
|
|
test_must_fail git worktree repair not-a-worktree >out 2>err &&
|
|
test_must_be_empty out &&
|
|
test_grep ".git is not a file" err
|
|
'
|
|
|
|
test_expect_success 'repo not found; .git not referencing repo' '
|
|
test_when_finished "rm -rf side not-a-repo && git worktree prune" &&
|
|
git worktree add --detach side &&
|
|
sed s,\.git/worktrees/side$,not-a-repo, side/.git >side/.newgit &&
|
|
mv side/.newgit side/.git &&
|
|
mkdir not-a-repo &&
|
|
test_must_fail git worktree repair side 2>err &&
|
|
test_grep ".git file does not reference a repository" err
|
|
'
|
|
|
|
test_expect_success 'repo not found; .git file broken' '
|
|
test_when_finished "rm -rf orig moved && git worktree prune" &&
|
|
git worktree add --detach orig &&
|
|
echo /invalid >orig/.git &&
|
|
mv orig moved &&
|
|
test_must_fail git worktree repair moved >out 2>err &&
|
|
test_must_be_empty out &&
|
|
test_grep ".git file broken" err
|
|
'
|
|
|
|
test_expect_success 'repair broken gitdir' '
|
|
test_when_finished "rm -rf orig moved && git worktree prune" &&
|
|
git worktree add --detach orig &&
|
|
sed s,orig/\.git$,moved/.git, .git/worktrees/orig/gitdir >expect &&
|
|
rm .git/worktrees/orig/gitdir &&
|
|
mv orig moved &&
|
|
git worktree repair moved 2>err &&
|
|
test_cmp expect .git/worktrees/orig/gitdir &&
|
|
test_grep "gitdir unreadable" err
|
|
'
|
|
|
|
test_expect_success 'repair incorrect gitdir' '
|
|
test_when_finished "rm -rf orig moved && git worktree prune" &&
|
|
git worktree add --detach orig &&
|
|
sed s,orig/\.git$,moved/.git, .git/worktrees/orig/gitdir >expect &&
|
|
mv orig moved &&
|
|
git worktree repair moved 2>err &&
|
|
test_cmp expect .git/worktrees/orig/gitdir &&
|
|
test_grep "gitdir incorrect" err
|
|
'
|
|
|
|
test_expect_success 'repair gitdir (implicit) from linked worktree' '
|
|
test_when_finished "rm -rf orig moved && git worktree prune" &&
|
|
git worktree add --detach orig &&
|
|
sed s,orig/\.git$,moved/.git, .git/worktrees/orig/gitdir >expect &&
|
|
mv orig moved &&
|
|
git -C moved worktree repair 2>err &&
|
|
test_cmp expect .git/worktrees/orig/gitdir &&
|
|
test_grep "gitdir incorrect" err
|
|
'
|
|
|
|
test_expect_success 'unable to repair gitdir (implicit) from main worktree' '
|
|
test_when_finished "rm -rf orig moved && git worktree prune" &&
|
|
git worktree add --detach orig &&
|
|
cat .git/worktrees/orig/gitdir >expect &&
|
|
mv orig moved &&
|
|
git worktree repair 2>err &&
|
|
test_cmp expect .git/worktrees/orig/gitdir &&
|
|
test_must_be_empty err
|
|
'
|
|
|
|
test_expect_success 'repair multiple gitdir files' '
|
|
test_when_finished "rm -rf orig1 orig2 moved1 moved2 &&
|
|
git worktree prune" &&
|
|
git worktree add --detach orig1 &&
|
|
git worktree add --detach orig2 &&
|
|
sed s,orig1/\.git$,moved1/.git, .git/worktrees/orig1/gitdir >expect1 &&
|
|
sed s,orig2/\.git$,moved2/.git, .git/worktrees/orig2/gitdir >expect2 &&
|
|
mv orig1 moved1 &&
|
|
mv orig2 moved2 &&
|
|
git worktree repair moved1 moved2 2>err &&
|
|
test_cmp expect1 .git/worktrees/orig1/gitdir &&
|
|
test_cmp expect2 .git/worktrees/orig2/gitdir &&
|
|
test_grep "gitdir incorrect:.*orig1/gitdir$" err &&
|
|
test_grep "gitdir incorrect:.*orig2/gitdir$" err
|
|
'
|
|
|
|
test_expect_success 'repair moved main and linked worktrees' '
|
|
test_when_finished "rm -rf main side mainmoved sidemoved" &&
|
|
test_create_repo main &&
|
|
test_commit -C main init &&
|
|
git -C main worktree add --detach ../side &&
|
|
sed "s,side/\.git$,sidemoved/.git," \
|
|
main/.git/worktrees/side/gitdir >expect-gitdir &&
|
|
sed "s,main/.git/worktrees/side$,mainmoved/.git/worktrees/side," \
|
|
side/.git >expect-gitfile &&
|
|
mv main mainmoved &&
|
|
mv side sidemoved &&
|
|
git -C mainmoved worktree repair ../sidemoved &&
|
|
test_cmp expect-gitdir mainmoved/.git/worktrees/side/gitdir &&
|
|
test_cmp expect-gitfile sidemoved/.git
|
|
'
|
|
|
|
test_expect_success 'repair copied main and linked worktrees' '
|
|
test_when_finished "rm -rf orig dup" &&
|
|
mkdir -p orig &&
|
|
git -C orig init main &&
|
|
test_commit -C orig/main nothing &&
|
|
git -C orig/main worktree add ../linked &&
|
|
cp orig/main/.git/worktrees/linked/gitdir orig/main.expect &&
|
|
cp orig/linked/.git orig/linked.expect &&
|
|
cp -R orig dup &&
|
|
sed "s,orig/linked/\.git$,dup/linked/.git," orig/main.expect >dup/main.expect &&
|
|
sed "s,orig/main/\.git/worktrees/linked$,dup/main/.git/worktrees/linked," \
|
|
orig/linked.expect >dup/linked.expect &&
|
|
git -C dup/main worktree repair ../linked &&
|
|
test_cmp orig/main.expect orig/main/.git/worktrees/linked/gitdir &&
|
|
test_cmp orig/linked.expect orig/linked/.git &&
|
|
test_cmp dup/main.expect dup/main/.git/worktrees/linked/gitdir &&
|
|
test_cmp dup/linked.expect dup/linked/.git
|
|
'
|
|
|
|
test_done
|