2014-11-30 09:24:47 +01:00
|
|
|
#!/bin/sh
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_description='test git worktree add'
|
2014-11-30 09:24:47 +01:00
|
|
|
|
2020-11-19 00:44:22 +01:00
|
|
|
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
|
tests: mark tests relying on the current default for `init.defaultBranch`
In addition to the manual adjustment to let the `linux-gcc` CI job run
the test suite with `master` and then with `main`, this patch makes sure
that GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME is set in all test scripts
that currently rely on the initial branch name being `master by default.
To determine which test scripts to mark up, the first step was to
force-set the default branch name to `master` in
- all test scripts that contain the keyword `master`,
- t4211, which expects `t/t4211/history.export` with a hard-coded ref to
initialize the default branch,
- t5560 because it sources `t/t556x_common` which uses `master`,
- t8002 and t8012 because both source `t/annotate-tests.sh` which also
uses `master`)
This trick was performed by this command:
$ sed -i '/^ *\. \.\/\(test-lib\|lib-\(bash\|cvs\|git-svn\)\|gitweb-lib\)\.sh$/i\
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=master\
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME\
' $(git grep -l master t/t[0-9]*.sh) \
t/t4211*.sh t/t5560*.sh t/t8002*.sh t/t8012*.sh
After that, careful, manual inspection revealed that some of the test
scripts containing the needle `master` do not actually rely on a
specific default branch name: either they mention `master` only in a
comment, or they initialize that branch specificially, or they do not
actually refer to the current default branch. Therefore, the
aforementioned modification was undone in those test scripts thusly:
$ git checkout HEAD -- \
t/t0027-auto-crlf.sh t/t0060-path-utils.sh \
t/t1011-read-tree-sparse-checkout.sh \
t/t1305-config-include.sh t/t1309-early-config.sh \
t/t1402-check-ref-format.sh t/t1450-fsck.sh \
t/t2024-checkout-dwim.sh \
t/t2106-update-index-assume-unchanged.sh \
t/t3040-subprojects-basic.sh t/t3301-notes.sh \
t/t3308-notes-merge.sh t/t3423-rebase-reword.sh \
t/t3436-rebase-more-options.sh \
t/t4015-diff-whitespace.sh t/t4257-am-interactive.sh \
t/t5323-pack-redundant.sh t/t5401-update-hooks.sh \
t/t5511-refspec.sh t/t5526-fetch-submodules.sh \
t/t5529-push-errors.sh t/t5530-upload-pack-error.sh \
t/t5548-push-porcelain.sh \
t/t5552-skipping-fetch-negotiator.sh \
t/t5572-pull-submodule.sh t/t5608-clone-2gb.sh \
t/t5614-clone-submodules-shallow.sh \
t/t7508-status.sh t/t7606-merge-custom.sh \
t/t9302-fast-import-unpack-limit.sh
We excluded one set of test scripts in these commands, though: the range
of `git p4` tests. The reason? `git p4` stores the (foreign) remote
branch in the branch called `p4/master`, which is obviously not the
default branch. Manual analysis revealed that only five of these tests
actually require a specific default branch name to pass; They were
modified thusly:
$ sed -i '/^ *\. \.\/lib-git-p4\.sh$/i\
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=master\
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME\
' t/t980[0167]*.sh t/t9811*.sh
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-11-19 00:44:19 +01:00
|
|
|
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
|
|
|
|
|
2014-11-30 09:24:47 +01:00
|
|
|
. ./test-lib.sh
|
|
|
|
|
2016-04-22 15:01:33 +02:00
|
|
|
. "$TEST_DIRECTORY"/lib-rebase.sh
|
|
|
|
|
2014-11-30 09:24:47 +01:00
|
|
|
test_expect_success 'setup' '
|
|
|
|
test_commit init
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" an existing worktree' '
|
2014-11-30 09:24:58 +01:00
|
|
|
mkdir -p existing/subtree &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git worktree add --detach existing main
|
2014-11-30 09:24:47 +01:00
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" an existing empty worktree' '
|
2014-11-30 09:24:58 +01:00
|
|
|
mkdir existing_empty &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --detach existing_empty main
|
2014-11-30 09:24:58 +01:00
|
|
|
'
|
|
|
|
|
2016-05-27 15:17:08 +02:00
|
|
|
test_expect_success '"add" using shorthand - fails when no previous branch' '
|
|
|
|
test_must_fail git worktree add existing_short -
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" using - shorthand' '
|
|
|
|
git checkout -b newbranch &&
|
|
|
|
echo hello >myworld &&
|
|
|
|
git add myworld &&
|
|
|
|
git commit -m myworld &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git checkout main &&
|
2016-05-27 15:17:08 +02:00
|
|
|
git worktree add short-hand - &&
|
|
|
|
echo refs/heads/newbranch >expect &&
|
|
|
|
git -C short-hand rev-parse --symbolic-full-name HEAD >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" refuses to checkout locked branch' '
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git worktree add zere main &&
|
2014-11-30 09:24:50 +01:00
|
|
|
! test -d zere &&
|
|
|
|
! test -d .git/worktrees/zere
|
|
|
|
'
|
|
|
|
|
2015-06-12 12:49:24 +02:00
|
|
|
test_expect_success 'checking out paths not complaining about linked checkouts' '
|
|
|
|
(
|
|
|
|
cd existing_empty &&
|
|
|
|
echo dirty >>init.t &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git checkout main -- init.t
|
2015-06-12 12:49:24 +02:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" worktree' '
|
2014-11-30 09:24:49 +01:00
|
|
|
git rev-parse HEAD >expect &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --detach here main &&
|
2014-11-30 09:24:47 +01:00
|
|
|
(
|
|
|
|
cd here &&
|
|
|
|
test_cmp ../init.t init.t &&
|
2014-11-30 09:24:49 +01:00
|
|
|
test_must_fail git symbolic-ref HEAD &&
|
|
|
|
git rev-parse HEAD >actual &&
|
|
|
|
test_cmp ../expect actual &&
|
2014-11-30 09:24:47 +01:00
|
|
|
git fsck
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2017-04-12 15:58:05 +02:00
|
|
|
test_expect_success '"add" worktree with lock' '
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --detach --lock here-with-lock main &&
|
2021-07-11 02:27:18 +02:00
|
|
|
test_when_finished "git worktree unlock here-with-lock || :" &&
|
2017-04-12 15:58:05 +02:00
|
|
|
test -f .git/worktrees/here-with-lock/locked
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" worktree from a subdir' '
|
2014-11-30 09:24:47 +01:00
|
|
|
(
|
|
|
|
mkdir sub &&
|
|
|
|
cd sub &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --detach here main &&
|
2014-11-30 09:24:47 +01:00
|
|
|
cd here &&
|
|
|
|
test_cmp ../../init.t init.t
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" from a linked checkout' '
|
2014-11-30 09:24:47 +01:00
|
|
|
(
|
|
|
|
cd here &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --detach nested-here main &&
|
2014-11-30 09:24:47 +01:00
|
|
|
cd nested-here &&
|
|
|
|
git fsck
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" worktree creating new branch' '
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add -b newmain there main &&
|
2014-11-30 09:24:47 +01:00
|
|
|
(
|
|
|
|
cd there &&
|
|
|
|
test_cmp ../init.t init.t &&
|
|
|
|
git symbolic-ref HEAD >actual &&
|
2020-11-19 00:44:22 +01:00
|
|
|
echo refs/heads/newmain >expect &&
|
2014-11-30 09:24:47 +01:00
|
|
|
test_cmp expect actual &&
|
|
|
|
git fsck
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2014-11-30 09:24:49 +01:00
|
|
|
test_expect_success 'die the same branch is already checked out' '
|
|
|
|
(
|
|
|
|
cd here &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git checkout newmain
|
2014-11-30 09:24:49 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2015-07-18 01:00:03 +02:00
|
|
|
test_expect_success SYMLINKS 'die the same branch is already checked out (symlink)' '
|
|
|
|
head=$(git -C there rev-parse --git-path HEAD) &&
|
|
|
|
ref=$(git -C there symbolic-ref HEAD) &&
|
|
|
|
rm "$head" &&
|
|
|
|
ln -s "$ref" "$head" &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git -C here checkout newmain
|
2015-07-18 01:00:03 +02:00
|
|
|
'
|
|
|
|
|
2015-01-03 10:41:26 +01:00
|
|
|
test_expect_success 'not die the same branch is already checked out' '
|
|
|
|
(
|
|
|
|
cd here &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --force anothernewmain newmain
|
2015-01-03 10:41:26 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2014-11-30 09:24:49 +01:00
|
|
|
test_expect_success 'not die on re-checking out current branch' '
|
|
|
|
(
|
|
|
|
cd there &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git checkout newmain
|
2014-11-30 09:24:49 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" from a bare repo' '
|
2014-11-30 09:24:56 +01:00
|
|
|
(
|
|
|
|
git clone --bare . bare &&
|
|
|
|
cd bare &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add -b bare-main ../there2 main
|
2014-11-30 09:24:56 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success 'checkout from a bare repo without "add"' '
|
2014-11-30 09:24:56 +01:00
|
|
|
(
|
|
|
|
cd bare &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git checkout main
|
2014-11-30 09:24:56 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2016-10-12 18:41:07 +02:00
|
|
|
test_expect_success '"add" default branch of a bare repo' '
|
|
|
|
(
|
|
|
|
git clone --bare . bare2 &&
|
|
|
|
cd bare2 &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add ../there3 main
|
2016-10-12 18:41:07 +02:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2014-11-30 09:24:57 +01:00
|
|
|
test_expect_success 'checkout with grafts' '
|
|
|
|
test_when_finished rm .git/info/grafts &&
|
|
|
|
test_commit abc &&
|
2015-12-22 16:05:52 +01:00
|
|
|
SHA1=$(git rev-parse HEAD) &&
|
2014-11-30 09:24:57 +01:00
|
|
|
test_commit def &&
|
|
|
|
test_commit xyz &&
|
2015-12-22 16:05:52 +01:00
|
|
|
echo "$(git rev-parse HEAD) $SHA1" >.git/info/grafts &&
|
2014-11-30 09:24:57 +01:00
|
|
|
cat >expected <<-\EOF &&
|
|
|
|
xyz
|
|
|
|
abc
|
|
|
|
EOF
|
|
|
|
git log --format=%s -2 >actual &&
|
|
|
|
test_cmp expected actual &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --detach grafted main &&
|
2014-11-30 09:24:57 +01:00
|
|
|
git --git-dir=grafted/.git log --format=%s -2 >actual &&
|
|
|
|
test_cmp expected actual
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:54 +02:00
|
|
|
test_expect_success '"add" from relative HEAD' '
|
2015-07-06 19:30:45 +02:00
|
|
|
test_commit a &&
|
|
|
|
test_commit b &&
|
|
|
|
test_commit c &&
|
|
|
|
git rev-parse HEAD~1 >expected &&
|
2015-07-06 19:30:54 +02:00
|
|
|
git worktree add relhead HEAD~1 &&
|
2015-07-06 19:30:45 +02:00
|
|
|
git -C relhead rev-parse HEAD >actual &&
|
|
|
|
test_cmp expected actual
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:58 +02:00
|
|
|
test_expect_success '"add -b" with <branch> omitted' '
|
|
|
|
git worktree add -b burble flornk &&
|
|
|
|
test_cmp_rev HEAD burble
|
|
|
|
'
|
|
|
|
|
2015-07-18 01:00:09 +02:00
|
|
|
test_expect_success '"add --detach" with <branch> omitted' '
|
|
|
|
git worktree add --detach fishhook &&
|
|
|
|
git rev-parse HEAD >expected &&
|
|
|
|
git -C fishhook rev-parse HEAD >actual &&
|
|
|
|
test_cmp expected actual &&
|
|
|
|
test_must_fail git -C fishhook symbolic-ref HEAD
|
|
|
|
'
|
|
|
|
|
2015-07-06 19:30:59 +02:00
|
|
|
test_expect_success '"add" with <branch> omitted' '
|
|
|
|
git worktree add wiffle/bat &&
|
|
|
|
test_cmp_rev HEAD bat
|
|
|
|
'
|
|
|
|
|
2018-04-24 23:56:35 +02:00
|
|
|
test_expect_success '"add" checks out existing branch of dwimd name' '
|
|
|
|
git branch dwim HEAD~1 &&
|
|
|
|
git worktree add dwim &&
|
|
|
|
test_cmp_rev HEAD~1 dwim &&
|
|
|
|
(
|
|
|
|
cd dwim &&
|
|
|
|
test_cmp_rev HEAD dwim
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add <path>" dwim fails with checked out branch' '
|
|
|
|
git checkout -b test-branch &&
|
|
|
|
test_must_fail git worktree add test-branch &&
|
|
|
|
test_path_is_missing test-branch
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add --force" with existing dwimd name doesnt die' '
|
|
|
|
git checkout test-branch &&
|
|
|
|
git worktree add --force test-branch
|
2015-07-06 19:30:59 +02:00
|
|
|
'
|
|
|
|
|
2015-07-18 01:00:09 +02:00
|
|
|
test_expect_success '"add" no auto-vivify with --detach and <branch> omitted' '
|
|
|
|
git worktree add --detach mish/mash &&
|
|
|
|
test_must_fail git rev-parse mash -- &&
|
|
|
|
test_must_fail git -C mish/mash symbolic-ref HEAD
|
|
|
|
'
|
|
|
|
|
2015-07-18 01:00:08 +02:00
|
|
|
test_expect_success '"add" -b/-B mutually exclusive' '
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git worktree add -b poodle -B poodle bamboo main
|
2015-07-18 01:00:08 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" -b/--detach mutually exclusive' '
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git worktree add -b poodle --detach bamboo main
|
2015-07-18 01:00:08 +02:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" -B/--detach mutually exclusive' '
|
2020-11-19 00:44:22 +01:00
|
|
|
test_must_fail git worktree add -B poodle --detach bamboo main
|
2015-07-18 01:00:08 +02:00
|
|
|
'
|
|
|
|
|
2016-02-15 14:35:33 +01:00
|
|
|
test_expect_success '"add -B" fails if the branch is checked out' '
|
2020-11-19 00:44:22 +01:00
|
|
|
git rev-parse newmain >before &&
|
|
|
|
test_must_fail git worktree add -B newmain bamboo main &&
|
|
|
|
git rev-parse newmain >after &&
|
2016-02-15 14:35:33 +01:00
|
|
|
test_cmp before after
|
|
|
|
'
|
|
|
|
|
2016-02-15 14:35:32 +01:00
|
|
|
test_expect_success 'add -B' '
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add -B poodle bamboo2 main^ &&
|
2016-02-15 14:35:32 +01:00
|
|
|
git -C bamboo2 symbolic-ref HEAD >actual &&
|
|
|
|
echo refs/heads/poodle >expected &&
|
|
|
|
test_cmp expected actual &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_cmp_rev main^ poodle
|
2016-02-15 14:35:32 +01:00
|
|
|
'
|
|
|
|
|
2018-08-15 22:56:30 +02:00
|
|
|
test_expect_success 'add --quiet' '
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --quiet another-worktree main 2>actual &&
|
2018-08-15 22:56:30 +02:00
|
|
|
test_must_be_empty actual
|
|
|
|
'
|
|
|
|
|
2015-09-28 15:06:15 +02:00
|
|
|
test_expect_success 'local clone from linked checkout' '
|
|
|
|
git clone --local here here-clone &&
|
|
|
|
( cd here-clone && git fsck )
|
|
|
|
'
|
|
|
|
|
2017-12-12 00:16:12 +01:00
|
|
|
test_expect_success 'local clone --shared from linked checkout' '
|
|
|
|
git -C bare worktree add --detach ../baretree &&
|
|
|
|
git clone --local --shared baretree bare-clone &&
|
|
|
|
grep /bare/ bare-clone/.git/objects/info/alternates
|
|
|
|
'
|
|
|
|
|
2016-03-29 12:11:01 +02:00
|
|
|
test_expect_success '"add" worktree with --no-checkout' '
|
|
|
|
git worktree add --no-checkout -b swamp swamp &&
|
|
|
|
! test -e swamp/init.t &&
|
|
|
|
git -C swamp reset --hard &&
|
|
|
|
test_cmp init.t swamp/init.t
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" worktree with --checkout' '
|
|
|
|
git worktree add --checkout -b swmap2 swamp2 &&
|
|
|
|
test_cmp init.t swamp2/init.t
|
|
|
|
'
|
|
|
|
|
2016-04-22 15:01:33 +02:00
|
|
|
test_expect_success 'put a worktree under rebase' '
|
|
|
|
git worktree add under-rebase &&
|
|
|
|
(
|
|
|
|
cd under-rebase &&
|
|
|
|
set_fake_editor &&
|
|
|
|
FAKE_LINES="edit 1" git rebase -i HEAD^ &&
|
|
|
|
git worktree list | grep "under-rebase.*detached HEAD"
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'add a worktree, checking out a rebased branch' '
|
|
|
|
test_must_fail git worktree add new-rebase under-rebase &&
|
|
|
|
! test -d new-rebase
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'checking out a rebased branch from another worktree' '
|
|
|
|
git worktree add new-place &&
|
|
|
|
test_must_fail git -C new-place checkout under-rebase
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'not allow to delete a branch under rebase' '
|
|
|
|
(
|
|
|
|
cd under-rebase &&
|
|
|
|
test_must_fail git branch -D under-rebase
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2016-04-22 15:01:36 +02:00
|
|
|
test_expect_success 'rename a branch under rebase not allowed' '
|
|
|
|
test_must_fail git branch -M under-rebase rebase-with-new-name
|
|
|
|
'
|
|
|
|
|
2016-04-22 15:01:33 +02:00
|
|
|
test_expect_success 'check out from current worktree branch ok' '
|
|
|
|
(
|
|
|
|
cd under-rebase &&
|
|
|
|
git checkout under-rebase &&
|
|
|
|
git checkout - &&
|
|
|
|
git rebase --abort
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2016-04-22 15:01:35 +02:00
|
|
|
test_expect_success 'checkout a branch under bisect' '
|
|
|
|
git worktree add under-bisect &&
|
|
|
|
(
|
|
|
|
cd under-bisect &&
|
|
|
|
git bisect start &&
|
|
|
|
git bisect bad &&
|
|
|
|
git bisect good HEAD~2 &&
|
|
|
|
git worktree list | grep "under-bisect.*detached HEAD" &&
|
|
|
|
test_must_fail git worktree add new-bisect under-bisect &&
|
|
|
|
! test -d new-bisect
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2016-04-22 15:01:36 +02:00
|
|
|
test_expect_success 'rename a branch under bisect not allowed' '
|
|
|
|
test_must_fail git branch -M under-bisect bisect-with-new-name
|
|
|
|
'
|
2017-11-26 20:43:53 +01:00
|
|
|
# Is branch "refs/heads/$1" set to pull from "$2/$3"?
|
|
|
|
test_branch_upstream () {
|
|
|
|
printf "%s\n" "$2" "refs/heads/$3" >expect.upstream &&
|
|
|
|
{
|
|
|
|
git config "branch.$1.remote" &&
|
|
|
|
git config "branch.$1.merge"
|
|
|
|
} >actual.upstream &&
|
|
|
|
test_cmp expect.upstream actual.upstream
|
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success '--track sets up tracking' '
|
|
|
|
test_when_finished rm -rf track &&
|
2020-11-19 00:44:22 +01:00
|
|
|
git worktree add --track -b track track main &&
|
|
|
|
test_branch_upstream track . main
|
2017-11-26 20:43:53 +01:00
|
|
|
'
|
|
|
|
|
|
|
|
# setup remote repository $1 and repository $2 with $1 set up as
|
2020-11-19 00:44:22 +01:00
|
|
|
# remote. The remote has two branches, main and foo.
|
2017-11-26 20:43:53 +01:00
|
|
|
setup_remote_repo () {
|
|
|
|
git init $1 &&
|
|
|
|
(
|
|
|
|
cd $1 &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_commit $1_main &&
|
2017-11-26 20:43:53 +01:00
|
|
|
git checkout -b foo &&
|
|
|
|
test_commit upstream_foo
|
|
|
|
) &&
|
|
|
|
git init $2 &&
|
|
|
|
(
|
|
|
|
cd $2 &&
|
2020-11-19 00:44:22 +01:00
|
|
|
test_commit $2_main &&
|
2017-11-26 20:43:53 +01:00
|
|
|
git remote add $1 ../$1 &&
|
|
|
|
git config remote.$1.fetch \
|
|
|
|
"refs/heads/*:refs/remotes/$1/*" &&
|
|
|
|
git fetch --all
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success '--no-track avoids setting up tracking' '
|
|
|
|
test_when_finished rm -rf repo_upstream repo_local foo &&
|
|
|
|
setup_remote_repo repo_upstream repo_local &&
|
|
|
|
(
|
|
|
|
cd repo_local &&
|
|
|
|
git worktree add --no-track -b foo ../foo repo_upstream/foo
|
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_must_fail git config "branch.foo.remote" &&
|
|
|
|
test_must_fail git config "branch.foo.merge" &&
|
|
|
|
test_cmp_rev refs/remotes/repo_upstream/foo refs/heads/foo
|
|
|
|
)
|
|
|
|
'
|
2016-04-22 15:01:36 +02:00
|
|
|
|
2017-11-26 20:43:54 +01:00
|
|
|
test_expect_success '"add" <path> <non-existent-branch> fails' '
|
|
|
|
test_must_fail git worktree add foo non-existent
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" <path> <branch> dwims' '
|
|
|
|
test_when_finished rm -rf repo_upstream repo_dwim foo &&
|
|
|
|
setup_remote_repo repo_upstream repo_dwim &&
|
|
|
|
git init repo_dwim &&
|
|
|
|
(
|
|
|
|
cd repo_dwim &&
|
|
|
|
git worktree add ../foo foo
|
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_branch_upstream foo repo_upstream foo &&
|
|
|
|
test_cmp_rev refs/remotes/repo_upstream/foo refs/heads/foo
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
checkout & worktree: introduce checkout.defaultRemote
Introduce a checkout.defaultRemote setting which can be used to
designate a remote to prefer (via checkout.defaultRemote=origin) when
running e.g. "git checkout master" to mean origin/master, even though
there's other remotes that have the "master" branch.
I want this because it's very handy to use this workflow to checkout a
repository and create a topic branch, then get back to a "master" as
retrieved from upstream:
(
cd /tmp &&
rm -rf tbdiff &&
git clone git@github.com:trast/tbdiff.git &&
cd tbdiff &&
git branch -m topic &&
git checkout master
)
That will output:
Branch 'master' set up to track remote branch 'master' from 'origin'.
Switched to a new branch 'master'
But as soon as a new remote is added (e.g. just to inspect something
from someone else) the DWIMery goes away:
(
cd /tmp &&
rm -rf tbdiff &&
git clone git@github.com:trast/tbdiff.git &&
cd tbdiff &&
git branch -m topic &&
git remote add avar git@github.com:avar/tbdiff.git &&
git fetch avar &&
git checkout master
)
Will output (without the advice output added earlier in this series):
error: pathspec 'master' did not match any file(s) known to git.
The new checkout.defaultRemote config allows me to say that whenever
that ambiguity comes up I'd like to prefer "origin", and it'll still
work as though the only remote I had was "origin".
Also adjust the advice.checkoutAmbiguousRemoteBranchName message to
mention this new config setting to the user, the full output on my
git.git is now (the last paragraph is new):
$ ./git --exec-path=$PWD checkout master
error: pathspec 'master' did not match any file(s) known to git.
hint: 'master' matched more than one remote tracking branch.
hint: We found 26 remotes with a reference that matched. So we fell back
hint: on trying to resolve the argument as a path, but failed there too!
hint:
hint: If you meant to check out a remote tracking branch on, e.g. 'origin',
hint: you can do so by fully qualifying the name with the --track option:
hint:
hint: git checkout --track origin/<name>
hint:
hint: If you'd like to always have checkouts of an ambiguous <name> prefer
hint: one remote, e.g. the 'origin' remote, consider setting
hint: checkout.defaultRemote=origin in your config.
I considered splitting this into checkout.defaultRemote and
worktree.defaultRemote, but it's probably less confusing to break our
own rules that anything shared between config should live in core.*
than have two config settings, and I couldn't come up with a short
name under core.* that made sense (core.defaultRemoteForCheckout?).
See also 70c9ac2f19 ("DWIM "git checkout frotz" to "git checkout -b
frotz origin/frotz"", 2009-10-18) which introduced this DWIM feature
to begin with, and 4e85333197 ("worktree: make add <path> <branch>
dwim", 2017-11-26) which added it to git-worktree.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-06-05 16:40:49 +02:00
|
|
|
test_expect_success '"add" <path> <branch> dwims with checkout.defaultRemote' '
|
|
|
|
test_when_finished rm -rf repo_upstream repo_dwim foo &&
|
|
|
|
setup_remote_repo repo_upstream repo_dwim &&
|
|
|
|
git init repo_dwim &&
|
|
|
|
(
|
|
|
|
cd repo_dwim &&
|
|
|
|
git remote add repo_upstream2 ../repo_upstream &&
|
|
|
|
git fetch repo_upstream2 &&
|
|
|
|
test_must_fail git worktree add ../foo foo &&
|
|
|
|
git -c checkout.defaultRemote=repo_upstream worktree add ../foo foo &&
|
|
|
|
git status -uno --porcelain >status.actual &&
|
2018-07-27 19:48:11 +02:00
|
|
|
test_must_be_empty status.actual
|
checkout & worktree: introduce checkout.defaultRemote
Introduce a checkout.defaultRemote setting which can be used to
designate a remote to prefer (via checkout.defaultRemote=origin) when
running e.g. "git checkout master" to mean origin/master, even though
there's other remotes that have the "master" branch.
I want this because it's very handy to use this workflow to checkout a
repository and create a topic branch, then get back to a "master" as
retrieved from upstream:
(
cd /tmp &&
rm -rf tbdiff &&
git clone git@github.com:trast/tbdiff.git &&
cd tbdiff &&
git branch -m topic &&
git checkout master
)
That will output:
Branch 'master' set up to track remote branch 'master' from 'origin'.
Switched to a new branch 'master'
But as soon as a new remote is added (e.g. just to inspect something
from someone else) the DWIMery goes away:
(
cd /tmp &&
rm -rf tbdiff &&
git clone git@github.com:trast/tbdiff.git &&
cd tbdiff &&
git branch -m topic &&
git remote add avar git@github.com:avar/tbdiff.git &&
git fetch avar &&
git checkout master
)
Will output (without the advice output added earlier in this series):
error: pathspec 'master' did not match any file(s) known to git.
The new checkout.defaultRemote config allows me to say that whenever
that ambiguity comes up I'd like to prefer "origin", and it'll still
work as though the only remote I had was "origin".
Also adjust the advice.checkoutAmbiguousRemoteBranchName message to
mention this new config setting to the user, the full output on my
git.git is now (the last paragraph is new):
$ ./git --exec-path=$PWD checkout master
error: pathspec 'master' did not match any file(s) known to git.
hint: 'master' matched more than one remote tracking branch.
hint: We found 26 remotes with a reference that matched. So we fell back
hint: on trying to resolve the argument as a path, but failed there too!
hint:
hint: If you meant to check out a remote tracking branch on, e.g. 'origin',
hint: you can do so by fully qualifying the name with the --track option:
hint:
hint: git checkout --track origin/<name>
hint:
hint: If you'd like to always have checkouts of an ambiguous <name> prefer
hint: one remote, e.g. the 'origin' remote, consider setting
hint: checkout.defaultRemote=origin in your config.
I considered splitting this into checkout.defaultRemote and
worktree.defaultRemote, but it's probably less confusing to break our
own rules that anything shared between config should live in core.*
than have two config settings, and I couldn't come up with a short
name under core.* that made sense (core.defaultRemoteForCheckout?).
See also 70c9ac2f19 ("DWIM "git checkout frotz" to "git checkout -b
frotz origin/frotz"", 2009-10-18) which introduced this DWIM feature
to begin with, and 4e85333197 ("worktree: make add <path> <branch>
dwim", 2017-11-26) which added it to git-worktree.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-06-05 16:40:49 +02:00
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_branch_upstream foo repo_upstream foo &&
|
|
|
|
test_cmp_rev refs/remotes/repo_upstream/foo refs/heads/foo
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2017-11-29 21:04:50 +01:00
|
|
|
test_expect_success 'git worktree add does not match remote' '
|
|
|
|
test_when_finished rm -rf repo_a repo_b foo &&
|
|
|
|
setup_remote_repo repo_a repo_b &&
|
|
|
|
(
|
|
|
|
cd repo_b &&
|
|
|
|
git worktree add ../foo
|
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_must_fail git config "branch.foo.remote" &&
|
|
|
|
test_must_fail git config "branch.foo.merge" &&
|
2019-11-13 00:07:45 +01:00
|
|
|
test_cmp_rev ! refs/remotes/repo_a/foo refs/heads/foo
|
2017-11-29 21:04:50 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'git worktree add --guess-remote sets up tracking' '
|
|
|
|
test_when_finished rm -rf repo_a repo_b foo &&
|
|
|
|
setup_remote_repo repo_a repo_b &&
|
|
|
|
(
|
|
|
|
cd repo_b &&
|
|
|
|
git worktree add --guess-remote ../foo
|
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_branch_upstream foo repo_a foo &&
|
|
|
|
test_cmp_rev refs/remotes/repo_a/foo refs/heads/foo
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2017-11-29 21:04:51 +01:00
|
|
|
test_expect_success 'git worktree add with worktree.guessRemote sets up tracking' '
|
|
|
|
test_when_finished rm -rf repo_a repo_b foo &&
|
|
|
|
setup_remote_repo repo_a repo_b &&
|
|
|
|
(
|
|
|
|
cd repo_b &&
|
|
|
|
git config worktree.guessRemote true &&
|
|
|
|
git worktree add ../foo
|
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_branch_upstream foo repo_a foo &&
|
|
|
|
test_cmp_rev refs/remotes/repo_a/foo refs/heads/foo
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'git worktree --no-guess-remote option overrides config' '
|
|
|
|
test_when_finished rm -rf repo_a repo_b foo &&
|
|
|
|
setup_remote_repo repo_a repo_b &&
|
|
|
|
(
|
|
|
|
cd repo_b &&
|
|
|
|
git config worktree.guessRemote true &&
|
|
|
|
git worktree add --no-guess-remote ../foo
|
|
|
|
) &&
|
|
|
|
(
|
|
|
|
cd foo &&
|
|
|
|
test_must_fail git config "branch.foo.remote" &&
|
|
|
|
test_must_fail git config "branch.foo.merge" &&
|
2019-11-13 00:07:45 +01:00
|
|
|
test_cmp_rev ! refs/remotes/repo_a/foo refs/heads/foo
|
2017-11-29 21:04:51 +01:00
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2017-12-07 22:20:17 +01:00
|
|
|
post_checkout_hook () {
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
gitdir=${1:-.git}
|
|
|
|
test_when_finished "rm -f $gitdir/hooks/post-checkout" &&
|
|
|
|
mkdir -p $gitdir/hooks &&
|
|
|
|
write_script $gitdir/hooks/post-checkout <<-\EOF
|
|
|
|
{
|
|
|
|
echo $*
|
|
|
|
git rev-parse --git-dir --show-toplevel
|
|
|
|
} >hook.actual
|
2017-12-07 22:20:17 +01:00
|
|
|
EOF
|
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success '"add" invokes post-checkout hook (branch)' '
|
|
|
|
post_checkout_hook &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
{
|
2018-05-13 04:24:13 +02:00
|
|
|
echo $ZERO_OID $(git rev-parse HEAD) 1 &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
echo $(pwd)/.git/worktrees/gumby &&
|
|
|
|
echo $(pwd)/gumby
|
|
|
|
} >hook.expect &&
|
2017-12-07 22:20:17 +01:00
|
|
|
git worktree add gumby &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
test_cmp hook.expect gumby/hook.actual
|
2017-12-07 22:20:17 +01:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" invokes post-checkout hook (detached)' '
|
|
|
|
post_checkout_hook &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
{
|
2018-05-13 04:24:13 +02:00
|
|
|
echo $ZERO_OID $(git rev-parse HEAD) 1 &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
echo $(pwd)/.git/worktrees/grumpy &&
|
|
|
|
echo $(pwd)/grumpy
|
|
|
|
} >hook.expect &&
|
2017-12-07 22:20:17 +01:00
|
|
|
git worktree add --detach grumpy &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
test_cmp hook.expect grumpy/hook.actual
|
2017-12-07 22:20:17 +01:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add --no-checkout" suppresses post-checkout hook' '
|
|
|
|
post_checkout_hook &&
|
|
|
|
rm -f hook.actual &&
|
|
|
|
git worktree add --no-checkout gloopy &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
test_path_is_missing gloopy/hook.actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" in other worktree invokes post-checkout hook' '
|
|
|
|
post_checkout_hook &&
|
|
|
|
{
|
2018-05-13 04:24:13 +02:00
|
|
|
echo $ZERO_OID $(git rev-parse HEAD) 1 &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
echo $(pwd)/.git/worktrees/guppy &&
|
|
|
|
echo $(pwd)/guppy
|
|
|
|
} >hook.expect &&
|
|
|
|
git -C gloopy worktree add --detach ../guppy &&
|
|
|
|
test_cmp hook.expect guppy/hook.actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" in bare repo invokes post-checkout hook' '
|
|
|
|
rm -rf bare &&
|
|
|
|
git clone --bare . bare &&
|
|
|
|
{
|
2018-05-13 04:24:13 +02:00
|
|
|
echo $ZERO_OID $(git --git-dir=bare rev-parse HEAD) 1 &&
|
worktree: add: fix 'post-checkout' not knowing new worktree location
Although "git worktree add" learned to run the 'post-checkout' hook in
ade546be47 (worktree: invoke post-checkout hook, 2017-12-07), it
neglected to change to the directory of the newly-created worktree
before running the hook. Instead, the hook runs within the directory
from which the "git worktree add" command itself was invoked, which
effectively neuters the hook since it knows nothing about the new
worktree directory.
Further, ade546be47 failed to sanitize the environment before running
the hook, which means that user-assigned values of GIT_DIR and
GIT_WORK_TREE could mislead the hook about the location of the new
worktree. In the case of "git worktree add" being run from a bare
repository, the GIT_DIR="." assigned by Git itself leaks into the hook's
environment and breaks Git commands; this is so even when the working
directory is correctly changed to the new worktree before the hook runs
since ".", relative to the new worktree directory, does not point at the
bare repository.
Fix these problems by (1) changing to the new worktree's directory
before running the hook, and (2) sanitizing the environment of GIT_DIR
and GIT_WORK_TREE so hooks can't be confused by misleading values.
Enhance the t2025 'post-checkout' tests to verify that the hook is
indeed run within the correct directory and that Git commands invoked by
the hook compute Git-dir and top-level worktree locations correctly.
While at it, also add two new tests: (1) verify that the hook is run
within the correct directory even when the new worktree is created from
a sibling worktree (as opposed to the main worktree); (2) verify that
the hook is provided with correct context when the new worktree is
created from a bare repository (test provided by Lars Schneider).
Implementation Notes:
Rather than sanitizing the environment of GIT_DIR and GIT_WORK_TREE, an
alternative would be to set them explicitly, as is already done for
other Git commands run internally by "git worktree add". This patch opts
instead to sanitize the environment in order to clearly document that
the worktree is fully functional by the time the hook is run, thus does
not require special environmental overrides.
The hook is run manually, rather than via run_hook_le(), since it needs
to change the working directory to that of the worktree, and
run_hook_le() does not provide such functionality. As this is a one-off
case, adding 'run_hook' overloads which allow the directory to be set
does not seem warranted at this time.
Reported-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2018-02-15 20:18:41 +01:00
|
|
|
echo $(pwd)/bare/worktrees/goozy &&
|
|
|
|
echo $(pwd)/goozy
|
|
|
|
} >hook.expect &&
|
|
|
|
post_checkout_hook bare &&
|
|
|
|
git -C bare worktree add --detach ../goozy &&
|
|
|
|
test_cmp hook.expect goozy/hook.actual
|
2017-12-07 22:20:17 +01:00
|
|
|
'
|
|
|
|
|
2018-08-28 23:20:22 +02:00
|
|
|
test_expect_success '"add" an existing but missing worktree' '
|
|
|
|
git worktree add --detach pneu &&
|
|
|
|
test_must_fail git worktree add --detach pneu &&
|
|
|
|
rm -fr pneu &&
|
2018-08-28 23:20:23 +02:00
|
|
|
test_must_fail git worktree add --detach pneu &&
|
|
|
|
git worktree add --force --detach pneu
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" an existing locked but missing worktree' '
|
|
|
|
git worktree add --detach gnoo &&
|
|
|
|
git worktree lock gnoo &&
|
|
|
|
test_when_finished "git worktree unlock gnoo || :" &&
|
|
|
|
rm -fr gnoo &&
|
|
|
|
test_must_fail git worktree add --detach gnoo &&
|
|
|
|
test_must_fail git worktree add --force --detach gnoo &&
|
|
|
|
git worktree add --force --force --detach gnoo
|
2018-08-28 23:20:22 +02:00
|
|
|
'
|
|
|
|
|
worktree: don't allow "add" validation to be fooled by suffix matching
"git worktree add <path>" performs various checks before approving
<path> as a valid location for the new worktree. Aside from ensuring
that <path> does not already exist, one of the questions it asks is
whether <path> is already a registered worktree. To perform this check,
it queries find_worktree() and disallows the "add" operation if
find_worktree() finds a match for <path>. As a convenience, however,
find_worktree() casts an overly wide net to allow users to identify
worktrees by shorthand in order to keep typing to a minimum. For
instance, it performs suffix matching which, given subtrees "foo/bar"
and "foo/baz", can correctly select the latter when asked only for
"baz".
"add" validation knows the exact path it is interrogating, so this sort
of heuristic-based matching is, at best, questionable for this use-case
and, at worst, may may accidentally interpret <path> as matching an
existing worktree and incorrectly report it as already registered even
when it isn't. (In fact, validate_worktree_add() already contains a
special case to avoid accidentally matching against the main worktree,
precisely due to this problem.)
Avoid the problem of potential accidental matching against an existing
worktree by instead taking advantage of find_worktree_by_path() which
matches paths deterministically, without applying any sort of magic
shorthand matching performed by find_worktree().
Reported-by: Cameron Gunnin <cameron.gunnin@synopsys.com>
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-02-24 10:08:48 +01:00
|
|
|
test_expect_success '"add" not tripped up by magic worktree matching"' '
|
|
|
|
# if worktree "sub1/bar" exists, "git worktree add bar" in distinct
|
|
|
|
# directory `sub2` should not mistakenly complain that `bar` is an
|
|
|
|
# already-registered worktree
|
|
|
|
mkdir sub1 sub2 &&
|
|
|
|
git -C sub1 --git-dir=../.git worktree add --detach bozo &&
|
|
|
|
git -C sub2 --git-dir=../.git worktree add --detach bozo
|
|
|
|
'
|
|
|
|
|
2019-03-08 10:28:34 +01:00
|
|
|
test_expect_success FUNNYNAMES 'sanitize generated worktree name' '
|
|
|
|
git worktree add --detach ". weird*..?.lock.lock" &&
|
|
|
|
test -d .git/worktrees/---weird-.-
|
|
|
|
'
|
|
|
|
|
2019-05-13 12:49:44 +02:00
|
|
|
test_expect_success '"add" should not fail because of another bad worktree' '
|
|
|
|
git init add-fail &&
|
|
|
|
(
|
|
|
|
cd add-fail &&
|
|
|
|
test_commit first &&
|
|
|
|
mkdir sub &&
|
|
|
|
git worktree add sub/to-be-deleted &&
|
|
|
|
rm -rf sub &&
|
|
|
|
git worktree add second
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2019-10-27 18:16:25 +01:00
|
|
|
test_expect_success '"add" with uninitialized submodule, with submodule.recurse unset' '
|
|
|
|
test_create_repo submodule &&
|
|
|
|
test_commit -C submodule first &&
|
|
|
|
test_create_repo project &&
|
|
|
|
git -C project submodule add ../submodule &&
|
|
|
|
git -C project add submodule &&
|
|
|
|
test_tick &&
|
|
|
|
git -C project commit -m add_sub &&
|
|
|
|
git clone project project-clone &&
|
|
|
|
git -C project-clone worktree add ../project-2
|
|
|
|
'
|
|
|
|
test_expect_success '"add" with uninitialized submodule, with submodule.recurse set' '
|
|
|
|
git -C project-clone -c submodule.recurse worktree add ../project-3
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" with initialized submodule, with submodule.recurse unset' '
|
|
|
|
git -C project-clone submodule update --init &&
|
|
|
|
git -C project-clone worktree add ../project-4
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '"add" with initialized submodule, with submodule.recurse set' '
|
|
|
|
git -C project-clone -c submodule.recurse worktree add ../project-5
|
|
|
|
'
|
|
|
|
|
2014-11-30 09:24:47 +01:00
|
|
|
test_done
|