2019-11-21 23:04:33 +01:00
|
|
|
#!/bin/sh
|
|
|
|
|
|
|
|
test_description='sparse checkout builtin tests'
|
|
|
|
|
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
|
|
|
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=master
|
|
|
|
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
|
|
|
|
|
2019-11-21 23:04:33 +01:00
|
|
|
. ./test-lib.sh
|
|
|
|
|
2019-12-20 20:41:14 +01:00
|
|
|
list_files() {
|
|
|
|
# Do not replace this with 'ls "$1"', as "ls" with BSD-lineage
|
|
|
|
# enables "-A" by default for root and ends up including ".git" and
|
|
|
|
# such in its output. (Note, though, that running the test suite as
|
|
|
|
# root is generally not recommended.)
|
|
|
|
(cd "$1" && printf '%s\n' *)
|
|
|
|
}
|
|
|
|
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files() {
|
|
|
|
list_files "$1" >actual &&
|
|
|
|
shift &&
|
|
|
|
printf "%s\n" $@ >expect &&
|
|
|
|
test_cmp expect actual
|
|
|
|
}
|
|
|
|
|
2019-11-21 23:04:33 +01:00
|
|
|
test_expect_success 'setup' '
|
|
|
|
git init repo &&
|
|
|
|
(
|
|
|
|
cd repo &&
|
|
|
|
echo "initial" >a &&
|
|
|
|
mkdir folder1 folder2 deep &&
|
|
|
|
mkdir deep/deeper1 deep/deeper2 &&
|
|
|
|
mkdir deep/deeper1/deepest &&
|
|
|
|
cp a folder1 &&
|
|
|
|
cp a folder2 &&
|
|
|
|
cp a deep &&
|
|
|
|
cp a deep/deeper1 &&
|
|
|
|
cp a deep/deeper2 &&
|
|
|
|
cp a deep/deeper1/deepest &&
|
|
|
|
git add . &&
|
|
|
|
git commit -m "initial commit"
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'git sparse-checkout list (empty)' '
|
|
|
|
git -C repo sparse-checkout list >list 2>err &&
|
|
|
|
test_must_be_empty list &&
|
|
|
|
test_i18ngrep "this worktree is not sparse (sparse-checkout file may not exist)" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'git sparse-checkout list (populated)' '
|
|
|
|
test_when_finished rm -f repo/.git/info/sparse-checkout &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/folder1/*
|
|
|
|
/deep/
|
|
|
|
**/a
|
|
|
|
!*bin*
|
2019-11-21 23:04:33 +01:00
|
|
|
EOF
|
|
|
|
cp repo/.git/info/sparse-checkout expect &&
|
|
|
|
git -C repo sparse-checkout list >list &&
|
|
|
|
test_cmp expect list
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:34 +01:00
|
|
|
test_expect_success 'git sparse-checkout init' '
|
|
|
|
git -C repo sparse-checkout init &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
2019-11-21 23:04:34 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
test_cmp_config -C repo true core.sparsecheckout &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a
|
2019-11-21 23:04:34 +01:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'git sparse-checkout list after init' '
|
|
|
|
git -C repo sparse-checkout list >actual &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
2019-11-21 23:04:34 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'init with existing sparse-checkout' '
|
|
|
|
echo "*folder*" >> repo/.git/info/sparse-checkout &&
|
|
|
|
git -C repo sparse-checkout init &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
*folder*
|
2019-11-21 23:04:34 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a folder1 folder2
|
2019-11-21 23:04:34 +01:00
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:35 +01:00
|
|
|
test_expect_success 'clone --sparse' '
|
2020-01-24 22:19:34 +01:00
|
|
|
git clone --sparse "file://$(pwd)/repo" clone &&
|
2019-11-21 23:04:35 +01:00
|
|
|
git -C clone sparse-checkout list >actual &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
2019-11-21 23:04:35 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect actual &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files clone a
|
2019-11-21 23:04:35 +01:00
|
|
|
'
|
|
|
|
|
sparse-checkout: avoid staging deletions of all files
sparse-checkout's purpose is to update the working tree to have it
reflect a subset of the tracked files. As such, it shouldn't be
switching branches, making commits, downloading or uploading data, or
staging or unstaging changes. Other than updating the worktree, the
only thing sparse-checkout should touch is the SKIP_WORKTREE bit of the
index. In particular, this sets up a nice invariant: running
sparse-checkout will never change the status of any file in `git status`
(reflecting the fact that we only set the SKIP_WORKTREE bit if the file
is safe to delete, i.e. if the file is unmodified).
Traditionally, we did a _really_ bad job with this goal. The
predecessor to sparse-checkout involved manual editing of
.git/info/sparse-checkout and running `git read-tree -mu HEAD`. That
command would stage and unstage changes and overwrite dirty changes in
the working tree.
The initial implementation of the sparse-checkout command was no better;
it simply invoked `git read-tree -mu HEAD` as a subprocess and had the
same caveats, though this issue came up repeatedly in review comments
and workarounds for the problems were put in place before the feature
was merged[1, 2, 3, 4, 5, 6; especially see 4 & 6].
[1] https://lore.kernel.org/git/CABPp-BFT9A5n=_bx5LsjCvbogqwSjiwgr5amcjgbU1iAk4KLJg@mail.gmail.com/
[2] https://lore.kernel.org/git/CABPp-BEmwSwg4tgJg6nVG8a3Hpn_g-=ZjApZF4EiJO+qVgu4uw@mail.gmail.com/
[3] https://lore.kernel.org/git/CABPp-BFV7TA0qwZCQpHCqx9N+JifyRyuBQ-pZ_oGfe-NOgyh7A@mail.gmail.com/
[4] https://lore.kernel.org/git/CABPp-BHYCCD+Vx5fq35jH82eHc1-P53Lz_aGNpHJNcx9kg2K-A@mail.gmail.com/
[5] https://lore.kernel.org/git/CABPp-BF+JWYZfDqp2Tn4AEKVp4b0YMA=Mbz4Nz62D-gGgiduYQ@mail.gmail.com/
[6] https://lore.kernel.org/git/20191121163706.GV23183@szeder.dev/
However, these workarounds, in addition to disabling the feature in a
number of important cases, also missed one special case. I'll get back
to it later.
In the 2.27.0 cycle, the disabling of the feature was lifted by finally
replacing the internal equivalent of `git read-tree -mu HEAD` with
something that did what we wanted: the new update_sparsity() function in
unpack-trees.c that only ever updates SKIP_WORKTREE bits in the index
and updates the working tree to match. This new function handles all
the cases that were problematic for the old implementation, except that
it breaks the same special case that avoided the workarounds of the old
implementation, but broke it in a different way.
So...that brings us to the special case: a git clone performed with
--no-checkout. As per the meaning of the flag, --no-checkout does not
check out any branch, with the implication that you aren't on one and
need to switch to one after the clone. Implementationally, HEAD is
still set (so in some sense you are partially on a branch), but
* the index is "unborn" (non-existent)
* there are no files in the working tree (other than .git/)
* the next time git switch (or git checkout) is run it will run
unpack_trees with `initial_checkout` flag set to true.
It is not until you run, e.g. `git switch <somebranch>` that the index
will be written and files in the working tree populated.
With this special --no-checkout case, the traditional `read-tree -mu
HEAD` behavior would have done the equivalent of acting like checkout --
switch to the default branch (HEAD), write out an index that matches
HEAD, and update the working tree to match. This special case slipped
through the avoid-making-changes checks in the original sparse-checkout
command and thus continued there.
After update_sparsity() was introduced and used (see commit f56f31af03
("sparse-checkout: use new update_sparsity() function", 2020-03-27)),
the behavior for the --no-checkout case changed: Due to git's
auto-vivification of an empty in-memory index (see do_read_index() and
note that `must_exist` is false), and due to sparse-checkout's
update_working_directory() code to always write out the index after it
was done, we got a new bug. That made it so that sparse-checkout would
switch the repository from a clone with an "unborn" index (i.e. still
needing an initial_checkout), to one that had a recorded index with no
entries. Thus, instead of all the files appearing deleted in `git
status` being known to git as a special artifact of not yet being on a
branch, our recording of an empty index made it suddenly look to git as
though it was definitely on a branch with ALL files staged for deletion!
A subsequent checkout or switch then had to contend with the fact that
it wasn't on an initial_checkout but had a bunch of staged deletions.
Make sure that sparse-checkout changes nothing in the index other than
the SKIP_WORKTREE bit; in particular, when the index is unborn we do not
have any branch checked out so there is no sparsification or
de-sparsification work to do. Simply return from
update_working_directory() early.
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-06-05 04:41:39 +02:00
|
|
|
test_expect_success 'interaction with clone --no-checkout (unborn index)' '
|
|
|
|
git clone --no-checkout "file://$(pwd)/repo" clone_no_checkout &&
|
|
|
|
git -C clone_no_checkout sparse-checkout init --cone &&
|
|
|
|
git -C clone_no_checkout sparse-checkout set folder1 &&
|
|
|
|
|
|
|
|
git -C clone_no_checkout sparse-checkout list >actual &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
folder1
|
|
|
|
EOF
|
|
|
|
test_cmp expect actual &&
|
|
|
|
|
|
|
|
# nothing checked out, expect "No such file or directory"
|
|
|
|
! ls clone_no_checkout/* >actual &&
|
|
|
|
test_must_be_empty actual &&
|
|
|
|
test_path_is_missing clone_no_checkout/.git/index &&
|
|
|
|
|
|
|
|
# No branch is checked out until we manually switch to one
|
|
|
|
git -C clone_no_checkout switch master &&
|
|
|
|
test_path_is_file clone_no_checkout/.git/index &&
|
|
|
|
check_files clone_no_checkout a folder1
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:36 +01:00
|
|
|
test_expect_success 'set enables config' '
|
|
|
|
git init empty-config &&
|
|
|
|
(
|
|
|
|
cd empty-config &&
|
|
|
|
test_commit test file &&
|
|
|
|
test_path_is_missing .git/config.worktree &&
|
2020-05-04 20:27:43 +02:00
|
|
|
git sparse-checkout set nothing &&
|
2019-11-21 23:04:36 +01:00
|
|
|
test_path_is_file .git/config.worktree &&
|
|
|
|
test_cmp_config true core.sparseCheckout
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'set sparse-checkout using builtin' '
|
|
|
|
git -C repo sparse-checkout set "/*" "!/*/" "*folder*" &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
*folder*
|
2019-11-21 23:04:36 +01:00
|
|
|
EOF
|
|
|
|
git -C repo sparse-checkout list >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a folder1 folder2
|
2019-11-21 23:04:36 +01:00
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:37 +01:00
|
|
|
test_expect_success 'set sparse-checkout using --stdin' '
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/folder1/
|
|
|
|
/folder2/
|
2019-11-21 23:04:37 +01:00
|
|
|
EOF
|
|
|
|
git -C repo sparse-checkout set --stdin <expect &&
|
|
|
|
git -C repo sparse-checkout list >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo "a folder1 folder2"
|
2019-11-21 23:04:37 +01:00
|
|
|
'
|
|
|
|
|
2020-02-11 16:02:23 +01:00
|
|
|
test_expect_success 'add to sparse-checkout' '
|
|
|
|
cat repo/.git/info/sparse-checkout >expect &&
|
|
|
|
cat >add <<-\EOF &&
|
|
|
|
pattern1
|
|
|
|
/folder1/
|
|
|
|
pattern2
|
|
|
|
EOF
|
|
|
|
cat add >>expect &&
|
|
|
|
git -C repo sparse-checkout add --stdin <add &&
|
|
|
|
git -C repo sparse-checkout list >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo "a folder1 folder2"
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:40 +01:00
|
|
|
test_expect_success 'cone mode: match patterns' '
|
|
|
|
git -C repo config --worktree core.sparseCheckoutCone true &&
|
|
|
|
rm -rf repo/a repo/folder1 repo/folder2 &&
|
2019-11-21 23:04:41 +01:00
|
|
|
git -C repo read-tree -mu HEAD 2>err &&
|
|
|
|
test_i18ngrep ! "disabling cone patterns" err &&
|
2019-11-21 23:04:40 +01:00
|
|
|
git -C repo reset --hard &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a folder1 folder2
|
2019-11-21 23:04:40 +01:00
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:41 +01:00
|
|
|
test_expect_success 'cone mode: warn on bad pattern' '
|
|
|
|
test_when_finished mv sparse-checkout repo/.git/info/ &&
|
|
|
|
cp repo/.git/info/sparse-checkout . &&
|
|
|
|
echo "!/deep/deeper/*" >>repo/.git/info/sparse-checkout &&
|
|
|
|
git -C repo read-tree -mu HEAD 2>err &&
|
|
|
|
test_i18ngrep "unrecognized negative pattern" err
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:38 +01:00
|
|
|
test_expect_success 'sparse-checkout disable' '
|
2019-11-21 23:04:47 +01:00
|
|
|
test_when_finished rm -rf repo/.git/info/sparse-checkout &&
|
2019-11-21 23:04:38 +01:00
|
|
|
git -C repo sparse-checkout disable &&
|
2019-11-21 23:04:47 +01:00
|
|
|
test_path_is_file repo/.git/info/sparse-checkout &&
|
2019-11-21 23:04:38 +01:00
|
|
|
git -C repo config --list >config &&
|
|
|
|
test_must_fail git config core.sparseCheckout &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a deep folder1 folder2
|
2019-11-21 23:04:38 +01:00
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:42 +01:00
|
|
|
test_expect_success 'cone mode: init and set' '
|
|
|
|
git -C repo sparse-checkout init --cone &&
|
|
|
|
git -C repo config --list >config &&
|
|
|
|
test_i18ngrep "core.sparsecheckoutcone=true" config &&
|
2019-12-20 20:41:14 +01:00
|
|
|
list_files repo >dir &&
|
2019-11-21 23:04:42 +01:00
|
|
|
echo a >expect &&
|
|
|
|
test_cmp expect dir &&
|
|
|
|
git -C repo sparse-checkout set deep/deeper1/deepest/ 2>err &&
|
|
|
|
test_must_be_empty err &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a deep &&
|
|
|
|
check_files repo/deep a deeper1 &&
|
|
|
|
check_files repo/deep/deeper1 a deepest &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
|
|
|
!/deep/*/
|
|
|
|
/deep/deeper1/
|
|
|
|
!/deep/deeper1/*/
|
|
|
|
/deep/deeper1/deepest/
|
2019-11-21 23:04:42 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
2020-01-24 22:19:32 +01:00
|
|
|
git -C repo sparse-checkout set --stdin 2>err <<-\EOF &&
|
|
|
|
folder1
|
|
|
|
folder2
|
2019-11-21 23:04:42 +01:00
|
|
|
EOF
|
|
|
|
test_must_be_empty err &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a folder1 folder2
|
2019-11-21 23:04:42 +01:00
|
|
|
'
|
|
|
|
|
2019-12-30 16:33:12 +01:00
|
|
|
test_expect_success 'cone mode: list' '
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
folder1
|
|
|
|
folder2
|
2019-12-30 16:33:12 +01:00
|
|
|
EOF
|
|
|
|
git -C repo sparse-checkout set --stdin <expect &&
|
|
|
|
git -C repo sparse-checkout list >actual 2>err &&
|
|
|
|
test_must_be_empty err &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:45 +01:00
|
|
|
test_expect_success 'cone mode: set with nested folders' '
|
|
|
|
git -C repo sparse-checkout set deep deep/deeper1/deepest 2>err &&
|
|
|
|
test_line_count = 0 err &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
2019-11-21 23:04:45 +01:00
|
|
|
EOF
|
|
|
|
test_cmp repo/.git/info/sparse-checkout expect
|
|
|
|
'
|
|
|
|
|
2020-02-11 16:02:23 +01:00
|
|
|
test_expect_success 'cone mode: add independent path' '
|
|
|
|
git -C repo sparse-checkout set deep/deeper1 &&
|
|
|
|
git -C repo sparse-checkout add folder1 &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
|
|
|
!/deep/*/
|
|
|
|
/deep/deeper1/
|
|
|
|
/folder1/
|
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo a deep folder1
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'cone mode: add sibling path' '
|
|
|
|
git -C repo sparse-checkout set deep/deeper1 &&
|
|
|
|
git -C repo sparse-checkout add deep/deeper2 &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
|
|
|
!/deep/*/
|
|
|
|
/deep/deeper1/
|
|
|
|
/deep/deeper2/
|
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo a deep
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'cone mode: add parent path' '
|
|
|
|
git -C repo sparse-checkout set deep/deeper1 folder1 &&
|
|
|
|
git -C repo sparse-checkout add deep &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
|
|
|
/folder1/
|
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo a deep folder1
|
|
|
|
'
|
|
|
|
|
2020-03-27 01:48:53 +01:00
|
|
|
test_expect_success 'not-up-to-date does not block rest of sparsification' '
|
2020-03-27 01:48:48 +01:00
|
|
|
test_when_finished git -C repo sparse-checkout disable &&
|
2019-11-21 23:04:51 +01:00
|
|
|
test_when_finished git -C repo reset --hard &&
|
2020-02-11 16:02:23 +01:00
|
|
|
git -C repo sparse-checkout set deep &&
|
2020-03-27 01:48:53 +01:00
|
|
|
|
2019-11-21 23:04:46 +01:00
|
|
|
echo update >repo/deep/deeper2/a &&
|
|
|
|
cp repo/.git/info/sparse-checkout expect &&
|
2020-03-27 01:48:53 +01:00
|
|
|
test_write_lines "!/deep/*/" "/deep/deeper1/" >>expect &&
|
|
|
|
|
|
|
|
git -C repo sparse-checkout set deep/deeper1 2>err &&
|
|
|
|
|
2020-03-27 01:48:58 +01:00
|
|
|
test_i18ngrep "The following paths are not up to date" err &&
|
2020-03-27 01:48:53 +01:00
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo/deep a deeper1 deeper2 &&
|
|
|
|
check_files repo/deep/deeper1 a deepest &&
|
|
|
|
check_files repo/deep/deeper1/deepest a &&
|
|
|
|
check_files repo/deep/deeper2 a
|
2019-11-21 23:04:46 +01:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'revert to old sparse-checkout on empty update' '
|
|
|
|
git init empty-test &&
|
|
|
|
(
|
|
|
|
echo >file &&
|
|
|
|
git add file &&
|
|
|
|
git commit -m "test" &&
|
2020-05-04 20:27:43 +02:00
|
|
|
git sparse-checkout set nothing 2>err &&
|
|
|
|
test_i18ngrep ! "Sparse checkout leaves no entry on working directory" err &&
|
2019-11-21 23:04:46 +01:00
|
|
|
test_i18ngrep ! ".git/index.lock" err &&
|
|
|
|
git sparse-checkout set file
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:48 +01:00
|
|
|
test_expect_success 'fail when lock is taken' '
|
|
|
|
test_when_finished rm -rf repo/.git/info/sparse-checkout.lock &&
|
|
|
|
touch repo/.git/info/sparse-checkout.lock &&
|
|
|
|
test_must_fail git -C repo sparse-checkout set deep 2>err &&
|
t1091: don't grep for `strerror()` string
We grep for "File exists" in stderr of the failing `git sparse-checkout`
to make sure that it failed for the right reason. We expect the string
to show up there since we call `strerror(errno)` in
`unable_to_lock_message()` in lockfile.c.
On the NonStop platform, this fails because the error string is "File
already exists", which doesn't match our grepping.
See 9042140097 ("test-dir-iterator: do not assume errno values",
2019-07-30) for a somewhat similar fix. There, we patched a test helper,
which meant we had access to `errno` and could investigate it better in
the test helper instead of just outputting the numerical value and
evaluating it in the test script. The current situation is different,
since (short of modifying the lockfile machinery, e.g., to be more
verbose) we don't have more than the output from `strerror()` available.
Except we do: We prefix `strerror(errno)` with `_("Unable to create
'%s.lock': ")`. Let's grep for that part instead. It verifies that we
were indeed unable to create the lock file. (If that fails for some
other reason than the file existing, we really really should expect
other tests to fail as well.)
An alternative fix would be to loosen the expression a bit and grep for
"File.* exists" instead. There would be no guarantee that some other
implementation couldn't come up with another error string, That is, that
could be the first move in an endless game of whack-a-mole. Of course,
it could also take us from "99" to "100" percent of the platforms and
we'd never have this problem again. But since we have another way of
addressing this, let's not even try the "loosen it up a bit" strategy.
Reported-by: Randall S. Becker <rsbecker@nexbridge.com>
Signed-off-by: Martin Ågren <martin.agren@gmail.com>
Acked-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-03-08 09:46:27 +01:00
|
|
|
test_i18ngrep "Unable to create .*\.lock" err
|
2019-11-21 23:04:48 +01:00
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:49 +01:00
|
|
|
test_expect_success '.gitignore should not warn about cone mode' '
|
|
|
|
git -C repo config --worktree core.sparseCheckoutCone true &&
|
|
|
|
echo "**/bin/*" >repo/.gitignore &&
|
|
|
|
git -C repo reset --hard 2>err &&
|
|
|
|
test_i18ngrep ! "disabling cone patterns" err
|
|
|
|
'
|
|
|
|
|
2020-03-27 01:48:53 +01:00
|
|
|
test_expect_success 'sparse-checkout (init|set|disable) warns with dirty status' '
|
2019-11-21 23:04:51 +01:00
|
|
|
git clone repo dirty &&
|
|
|
|
echo dirty >dirty/folder1/a &&
|
2020-03-27 01:48:53 +01:00
|
|
|
|
|
|
|
git -C dirty sparse-checkout init 2>err &&
|
2020-03-27 01:48:58 +01:00
|
|
|
test_i18ngrep "warning.*The following paths are not up to date" err &&
|
2020-03-27 01:48:53 +01:00
|
|
|
|
|
|
|
git -C dirty sparse-checkout set /folder2/* /deep/deeper1/* 2>err &&
|
2020-03-27 01:48:58 +01:00
|
|
|
test_i18ngrep "warning.*The following paths are not up to date" err &&
|
2020-03-27 01:48:53 +01:00
|
|
|
test_path_is_file dirty/folder1/a &&
|
|
|
|
|
|
|
|
git -C dirty sparse-checkout disable 2>err &&
|
|
|
|
test_must_be_empty err &&
|
|
|
|
|
2019-11-21 23:04:51 +01:00
|
|
|
git -C dirty reset --hard &&
|
|
|
|
git -C dirty sparse-checkout init &&
|
|
|
|
git -C dirty sparse-checkout set /folder2/* /deep/deeper1/* &&
|
2020-03-27 01:48:53 +01:00
|
|
|
test_path_is_missing dirty/folder1/a &&
|
|
|
|
git -C dirty sparse-checkout disable &&
|
|
|
|
test_path_is_file dirty/folder1/a
|
2019-11-21 23:04:51 +01:00
|
|
|
'
|
|
|
|
|
2020-03-27 01:48:59 +01:00
|
|
|
test_expect_success 'sparse-checkout (init|set|disable) warns with unmerged status' '
|
|
|
|
git clone repo unmerged &&
|
|
|
|
|
|
|
|
cat >input <<-EOF &&
|
2020-07-30 01:13:51 +02:00
|
|
|
0 $ZERO_OID folder1/a
|
2020-03-27 01:48:59 +01:00
|
|
|
100644 $(git -C unmerged rev-parse HEAD:folder1/a) 1 folder1/a
|
|
|
|
EOF
|
|
|
|
git -C unmerged update-index --index-info <input &&
|
|
|
|
|
|
|
|
git -C unmerged sparse-checkout init 2>err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
|
|
|
|
git -C unmerged sparse-checkout set /folder2/* /deep/deeper1/* 2>err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
test_path_is_file dirty/folder1/a &&
|
|
|
|
|
|
|
|
git -C unmerged sparse-checkout disable 2>err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
|
|
|
|
git -C unmerged reset --hard &&
|
|
|
|
git -C unmerged sparse-checkout init &&
|
|
|
|
git -C unmerged sparse-checkout set /folder2/* /deep/deeper1/* &&
|
|
|
|
git -C unmerged sparse-checkout disable
|
|
|
|
'
|
|
|
|
|
2020-03-27 01:49:01 +01:00
|
|
|
test_expect_success 'sparse-checkout reapply' '
|
|
|
|
git clone repo tweak &&
|
|
|
|
|
|
|
|
echo dirty >tweak/deep/deeper2/a &&
|
|
|
|
|
|
|
|
cat >input <<-EOF &&
|
2020-07-30 01:13:51 +02:00
|
|
|
0 $ZERO_OID folder1/a
|
2020-03-27 01:49:01 +01:00
|
|
|
100644 $(git -C tweak rev-parse HEAD:folder1/a) 1 folder1/a
|
|
|
|
EOF
|
|
|
|
git -C tweak update-index --index-info <input &&
|
|
|
|
|
|
|
|
git -C tweak sparse-checkout init --cone 2>err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are not up to date" err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
|
|
|
|
git -C tweak sparse-checkout set folder2 deep/deeper1 2>err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are not up to date" err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
|
|
|
|
git -C tweak sparse-checkout reapply 2>err &&
|
|
|
|
test_i18ngrep "warning.*The following paths are not up to date" err &&
|
|
|
|
test_path_is_file tweak/deep/deeper2/a &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
test_path_is_file tweak/folder1/a &&
|
|
|
|
|
|
|
|
git -C tweak checkout HEAD deep/deeper2/a &&
|
|
|
|
git -C tweak sparse-checkout reapply 2>err &&
|
|
|
|
test_i18ngrep ! "warning.*The following paths are not up to date" err &&
|
|
|
|
test_path_is_missing tweak/deep/deeper2/a &&
|
|
|
|
test_i18ngrep "warning.*The following paths are unmerged" err &&
|
|
|
|
test_path_is_file tweak/folder1/a &&
|
|
|
|
|
|
|
|
git -C tweak add folder1/a &&
|
|
|
|
git -C tweak sparse-checkout reapply 2>err &&
|
|
|
|
test_must_be_empty err &&
|
|
|
|
test_path_is_missing tweak/deep/deeper2/a &&
|
|
|
|
test_path_is_missing tweak/folder1/a &&
|
|
|
|
|
|
|
|
git -C tweak sparse-checkout disable
|
|
|
|
'
|
|
|
|
|
sparse-checkout: respect core.ignoreCase in cone mode
When a user uses the sparse-checkout feature in cone mode, they
add patterns using "git sparse-checkout set <dir1> <dir2> ..."
or by using "--stdin" to provide the directories line-by-line over
stdin. This behaviour naturally looks a lot like the way a user
would type "git add <dir1> <dir2> ..."
If core.ignoreCase is enabled, then "git add" will match the input
using a case-insensitive match. Do the same for the sparse-checkout
feature.
Perform case-insensitive checks while updating the skip-worktree
bits during unpack_trees(). This is done by changing the hash
algorithm and hashmap comparison methods to optionally use case-
insensitive methods.
When this is enabled, there is a small performance cost in the
hashing algorithm. To tease out the worst possible case, the
following was run on a repo with a deep directory structure:
git ls-tree -d -r --name-only HEAD |
git sparse-checkout set --stdin
The 'set' command was timed with core.ignoreCase disabled or
enabled. For the repo with a deep history, the numbers were
core.ignoreCase=false: 62s
core.ignoreCase=true: 74s (+19.3%)
For reproducibility, the equivalent test on the Linux kernel
repository had these numbers:
core.ignoreCase=false: 3.1s
core.ignoreCase=true: 3.6s (+16%)
Now, this is not an entirely fair comparison, as most users
will define their sparse cone using more shallow directories,
and the performance improvement from eb42feca97 ("unpack-trees:
hash less in cone mode" 2019-11-21) can remove most of the
hash cost. For a more realistic test, drop the "-r" from the
ls-tree command to store only the first-level directories.
In that case, the Linux kernel repository takes 0.2-0.25s in
each case, and the deep repository takes one second, plus or
minus 0.05s, in each case.
Thus, we _can_ demonstrate a cost to this change, but it is
unlikely to matter to any reasonable sparse-checkout cone.
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-12-13 19:09:53 +01:00
|
|
|
test_expect_success 'cone mode: set with core.ignoreCase=true' '
|
2020-03-27 01:48:48 +01:00
|
|
|
rm repo/.git/info/sparse-checkout &&
|
sparse-checkout: respect core.ignoreCase in cone mode
When a user uses the sparse-checkout feature in cone mode, they
add patterns using "git sparse-checkout set <dir1> <dir2> ..."
or by using "--stdin" to provide the directories line-by-line over
stdin. This behaviour naturally looks a lot like the way a user
would type "git add <dir1> <dir2> ..."
If core.ignoreCase is enabled, then "git add" will match the input
using a case-insensitive match. Do the same for the sparse-checkout
feature.
Perform case-insensitive checks while updating the skip-worktree
bits during unpack_trees(). This is done by changing the hash
algorithm and hashmap comparison methods to optionally use case-
insensitive methods.
When this is enabled, there is a small performance cost in the
hashing algorithm. To tease out the worst possible case, the
following was run on a repo with a deep directory structure:
git ls-tree -d -r --name-only HEAD |
git sparse-checkout set --stdin
The 'set' command was timed with core.ignoreCase disabled or
enabled. For the repo with a deep history, the numbers were
core.ignoreCase=false: 62s
core.ignoreCase=true: 74s (+19.3%)
For reproducibility, the equivalent test on the Linux kernel
repository had these numbers:
core.ignoreCase=false: 3.1s
core.ignoreCase=true: 3.6s (+16%)
Now, this is not an entirely fair comparison, as most users
will define their sparse cone using more shallow directories,
and the performance improvement from eb42feca97 ("unpack-trees:
hash less in cone mode" 2019-11-21) can remove most of the
hash cost. For a more realistic test, drop the "-r" from the
ls-tree command to store only the first-level directories.
In that case, the Linux kernel repository takes 0.2-0.25s in
each case, and the deep repository takes one second, plus or
minus 0.05s, in each case.
Thus, we _can_ demonstrate a cost to this change, but it is
unlikely to matter to any reasonable sparse-checkout cone.
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-12-13 19:09:53 +01:00
|
|
|
git -C repo sparse-checkout init --cone &&
|
|
|
|
git -C repo -c core.ignoreCase=true sparse-checkout set folder1 &&
|
2020-01-24 22:19:32 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/folder1/
|
sparse-checkout: respect core.ignoreCase in cone mode
When a user uses the sparse-checkout feature in cone mode, they
add patterns using "git sparse-checkout set <dir1> <dir2> ..."
or by using "--stdin" to provide the directories line-by-line over
stdin. This behaviour naturally looks a lot like the way a user
would type "git add <dir1> <dir2> ..."
If core.ignoreCase is enabled, then "git add" will match the input
using a case-insensitive match. Do the same for the sparse-checkout
feature.
Perform case-insensitive checks while updating the skip-worktree
bits during unpack_trees(). This is done by changing the hash
algorithm and hashmap comparison methods to optionally use case-
insensitive methods.
When this is enabled, there is a small performance cost in the
hashing algorithm. To tease out the worst possible case, the
following was run on a repo with a deep directory structure:
git ls-tree -d -r --name-only HEAD |
git sparse-checkout set --stdin
The 'set' command was timed with core.ignoreCase disabled or
enabled. For the repo with a deep history, the numbers were
core.ignoreCase=false: 62s
core.ignoreCase=true: 74s (+19.3%)
For reproducibility, the equivalent test on the Linux kernel
repository had these numbers:
core.ignoreCase=false: 3.1s
core.ignoreCase=true: 3.6s (+16%)
Now, this is not an entirely fair comparison, as most users
will define their sparse cone using more shallow directories,
and the performance improvement from eb42feca97 ("unpack-trees:
hash less in cone mode" 2019-11-21) can remove most of the
hash cost. For a more realistic test, drop the "-r" from the
ls-tree command to store only the first-level directories.
In that case, the Linux kernel repository takes 0.2-0.25s in
each case, and the deep repository takes one second, plus or
minus 0.05s, in each case.
Thus, we _can_ demonstrate a cost to this change, but it is
unlikely to matter to any reasonable sparse-checkout cone.
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-12-13 19:09:53 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files repo a folder1
|
sparse-checkout: respect core.ignoreCase in cone mode
When a user uses the sparse-checkout feature in cone mode, they
add patterns using "git sparse-checkout set <dir1> <dir2> ..."
or by using "--stdin" to provide the directories line-by-line over
stdin. This behaviour naturally looks a lot like the way a user
would type "git add <dir1> <dir2> ..."
If core.ignoreCase is enabled, then "git add" will match the input
using a case-insensitive match. Do the same for the sparse-checkout
feature.
Perform case-insensitive checks while updating the skip-worktree
bits during unpack_trees(). This is done by changing the hash
algorithm and hashmap comparison methods to optionally use case-
insensitive methods.
When this is enabled, there is a small performance cost in the
hashing algorithm. To tease out the worst possible case, the
following was run on a repo with a deep directory structure:
git ls-tree -d -r --name-only HEAD |
git sparse-checkout set --stdin
The 'set' command was timed with core.ignoreCase disabled or
enabled. For the repo with a deep history, the numbers were
core.ignoreCase=false: 62s
core.ignoreCase=true: 74s (+19.3%)
For reproducibility, the equivalent test on the Linux kernel
repository had these numbers:
core.ignoreCase=false: 3.1s
core.ignoreCase=true: 3.6s (+16%)
Now, this is not an entirely fair comparison, as most users
will define their sparse cone using more shallow directories,
and the performance improvement from eb42feca97 ("unpack-trees:
hash less in cone mode" 2019-11-21) can remove most of the
hash cost. For a more realistic test, drop the "-r" from the
ls-tree command to store only the first-level directories.
In that case, the Linux kernel repository takes 0.2-0.25s in
each case, and the deep repository takes one second, plus or
minus 0.05s, in each case.
Thus, we _can_ demonstrate a cost to this change, but it is
unlikely to matter to any reasonable sparse-checkout cone.
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2019-12-13 19:09:53 +01:00
|
|
|
'
|
|
|
|
|
2019-12-30 16:33:13 +01:00
|
|
|
test_expect_success 'interaction with submodules' '
|
|
|
|
git clone repo super &&
|
|
|
|
(
|
|
|
|
cd super &&
|
|
|
|
mkdir modules &&
|
|
|
|
git submodule add ../repo modules/child &&
|
|
|
|
git add . &&
|
|
|
|
git commit -m "add submodule" &&
|
|
|
|
git sparse-checkout init --cone &&
|
|
|
|
git sparse-checkout set folder1
|
|
|
|
) &&
|
2020-01-24 22:19:31 +01:00
|
|
|
check_files super a folder1 modules &&
|
|
|
|
check_files super/modules/child a deep folder1 folder2
|
2019-12-30 16:33:13 +01:00
|
|
|
'
|
|
|
|
|
2020-01-24 22:19:33 +01:00
|
|
|
test_expect_success 'different sparse-checkouts with worktrees' '
|
|
|
|
git -C repo worktree add --detach ../worktree &&
|
|
|
|
check_files worktree "a deep folder1 folder2" &&
|
|
|
|
git -C worktree sparse-checkout init --cone &&
|
|
|
|
git -C repo sparse-checkout set folder1 &&
|
|
|
|
git -C worktree sparse-checkout set deep/deeper1 &&
|
|
|
|
check_files repo a folder1 &&
|
|
|
|
check_files worktree a deep
|
|
|
|
'
|
|
|
|
|
2020-01-31 21:16:15 +01:00
|
|
|
test_expect_success 'set using filename keeps file on-disk' '
|
|
|
|
git -C repo sparse-checkout set a deep &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/a/
|
|
|
|
/deep/
|
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo a deep
|
|
|
|
'
|
|
|
|
|
2020-01-24 22:19:36 +01:00
|
|
|
check_read_tree_errors () {
|
|
|
|
REPO=$1
|
|
|
|
FILES=$2
|
|
|
|
ERRORS=$3
|
2020-01-31 21:16:10 +01:00
|
|
|
git -C $REPO -c core.sparseCheckoutCone=false read-tree -mu HEAD 2>err &&
|
|
|
|
test_must_be_empty err &&
|
|
|
|
check_files $REPO "$FILES" &&
|
2020-01-24 22:19:36 +01:00
|
|
|
git -C $REPO read-tree -mu HEAD 2>err &&
|
|
|
|
if test -z "$ERRORS"
|
|
|
|
then
|
|
|
|
test_must_be_empty err
|
|
|
|
else
|
|
|
|
test_i18ngrep "$ERRORS" err
|
|
|
|
fi &&
|
|
|
|
check_files $REPO $FILES
|
|
|
|
}
|
|
|
|
|
|
|
|
test_expect_success 'pattern-checks: /A/**' '
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/folder1/**
|
|
|
|
EOF
|
|
|
|
check_read_tree_errors repo "a folder1" "disabling cone pattern matching"
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'pattern-checks: /A/**/B/' '
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/**/deepest
|
|
|
|
EOF
|
|
|
|
check_read_tree_errors repo "a deep" "disabling cone pattern matching" &&
|
|
|
|
check_files repo/deep "deeper1" &&
|
|
|
|
check_files repo/deep/deeper1 "deepest"
|
|
|
|
'
|
|
|
|
|
2020-01-24 22:19:37 +01:00
|
|
|
test_expect_success 'pattern-checks: too short' '
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
sparse-checkout: allow one-character directories in cone mode
In 9e6d3e64 (sparse-checkout: detect short patterns, 2020-01-24), a
condition on the minimum length of a cone-mode pattern was introduced.
However, this condition was off-by-one.
If we have a directory with a single character, say "b", then the
command
git sparse-checkout set b
will correctly add the pattern "/b/" to the sparse-checkout file. When
this is interpeted in dir.c, the pattern is "/b" with the
PATTERN_FLAG_MUSTBEDIR flag. This string has length two, which satisfies
our inclusive inequality (<= 2).
The reason for this inequality is that we will start to read the pattern
string character-by-character using three char pointers: prev, cur,
next. In particular, next is set to the current pattern plus two. The
mistake was that next will still be a valid pointer when the pattern
length is two, since the string is null-terminated.
Make this inequality strict so these patterns work.
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-02-20 21:07:06 +01:00
|
|
|
/
|
2020-01-24 22:19:37 +01:00
|
|
|
EOF
|
|
|
|
check_read_tree_errors repo "a" "disabling cone pattern matching"
|
|
|
|
'
|
sparse-checkout: allow one-character directories in cone mode
In 9e6d3e64 (sparse-checkout: detect short patterns, 2020-01-24), a
condition on the minimum length of a cone-mode pattern was introduced.
However, this condition was off-by-one.
If we have a directory with a single character, say "b", then the
command
git sparse-checkout set b
will correctly add the pattern "/b/" to the sparse-checkout file. When
this is interpeted in dir.c, the pattern is "/b" with the
PATTERN_FLAG_MUSTBEDIR flag. This string has length two, which satisfies
our inclusive inequality (<= 2).
The reason for this inequality is that we will start to read the pattern
string character-by-character using three char pointers: prev, cur,
next. In particular, next is set to the current pattern plus two. The
mistake was that next will still be a valid pointer when the pattern
length is two, since the string is null-terminated.
Make this inequality strict so these patterns work.
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2020-02-20 21:07:06 +01:00
|
|
|
test_expect_success 'pattern-checks: not too short' '
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/b/
|
|
|
|
EOF
|
|
|
|
git -C repo read-tree -mu HEAD 2>err &&
|
|
|
|
test_must_be_empty err &&
|
|
|
|
check_files repo a
|
|
|
|
'
|
2020-01-24 22:19:37 +01:00
|
|
|
|
2020-01-31 21:16:08 +01:00
|
|
|
test_expect_success 'pattern-checks: trailing "*"' '
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/a*
|
|
|
|
EOF
|
|
|
|
check_read_tree_errors repo "a" "disabling cone pattern matching"
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'pattern-checks: starting "*"' '
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
*eep/
|
|
|
|
EOF
|
|
|
|
check_read_tree_errors repo "a deep" "disabling cone pattern matching"
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'pattern-checks: contained glob characters' '
|
|
|
|
for c in "[a]" "\\" "?" "*"
|
|
|
|
do
|
|
|
|
cat >repo/.git/info/sparse-checkout <<-EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
something$c-else/
|
|
|
|
EOF
|
|
|
|
check_read_tree_errors repo "a" "disabling cone pattern matching"
|
|
|
|
done
|
|
|
|
'
|
|
|
|
|
2020-01-31 21:16:13 +01:00
|
|
|
test_expect_success BSLASHPSPEC 'pattern-checks: escaped characters' '
|
2020-01-31 21:16:09 +01:00
|
|
|
git clone repo escaped &&
|
|
|
|
TREEOID=$(git -C escaped rev-parse HEAD:folder1) &&
|
|
|
|
NEWTREE=$(git -C escaped mktree <<-EOF
|
|
|
|
$(git -C escaped ls-tree HEAD)
|
|
|
|
040000 tree $TREEOID zbad\\dir
|
|
|
|
040000 tree $TREEOID zdoes*exist
|
2020-01-31 21:16:13 +01:00
|
|
|
040000 tree $TREEOID zglob[!a]?
|
2020-01-31 21:16:09 +01:00
|
|
|
EOF
|
|
|
|
) &&
|
|
|
|
COMMIT=$(git -C escaped commit-tree $NEWTREE -p HEAD) &&
|
|
|
|
git -C escaped reset --hard $COMMIT &&
|
2020-01-31 21:16:13 +01:00
|
|
|
check_files escaped "a deep folder1 folder2 zbad\\dir zdoes*exist" zglob[!a]? &&
|
2020-01-31 21:16:09 +01:00
|
|
|
git -C escaped sparse-checkout init --cone &&
|
2020-01-31 21:16:13 +01:00
|
|
|
git -C escaped sparse-checkout set zbad\\dir/bogus "zdoes*not*exist" "zdoes*exist" "zglob[!a]?" &&
|
2020-01-31 21:16:10 +01:00
|
|
|
cat >expect <<-\EOF &&
|
2020-01-31 21:16:08 +01:00
|
|
|
/*
|
|
|
|
!/*/
|
2020-01-31 21:16:09 +01:00
|
|
|
/zbad\\dir/
|
|
|
|
!/zbad\\dir/*/
|
2020-01-31 21:16:10 +01:00
|
|
|
/zbad\\dir/bogus/
|
2020-01-31 21:16:09 +01:00
|
|
|
/zdoes\*exist/
|
2020-01-31 21:16:10 +01:00
|
|
|
/zdoes\*not\*exist/
|
2020-01-31 21:16:13 +01:00
|
|
|
/zglob\[!a]\?/
|
2020-01-31 21:16:08 +01:00
|
|
|
EOF
|
2020-01-31 21:16:10 +01:00
|
|
|
test_cmp expect escaped/.git/info/sparse-checkout &&
|
2020-01-31 21:16:13 +01:00
|
|
|
check_read_tree_errors escaped "a zbad\\dir zdoes*exist zglob[!a]?" &&
|
2020-01-31 21:16:12 +01:00
|
|
|
git -C escaped ls-tree -d --name-only HEAD >list-expect &&
|
|
|
|
git -C escaped sparse-checkout set --stdin <list-expect &&
|
2020-01-31 21:16:11 +01:00
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
|
|
|
/folder1/
|
|
|
|
/folder2/
|
|
|
|
/zbad\\dir/
|
|
|
|
/zdoes\*exist/
|
2020-01-31 21:16:13 +01:00
|
|
|
/zglob\[!a]\?/
|
2020-01-31 21:16:11 +01:00
|
|
|
EOF
|
|
|
|
test_cmp expect escaped/.git/info/sparse-checkout &&
|
2020-01-31 21:16:13 +01:00
|
|
|
check_files escaped "a deep folder1 folder2 zbad\\dir zdoes*exist" zglob[!a]? &&
|
2020-01-31 21:16:12 +01:00
|
|
|
git -C escaped sparse-checkout list >list-actual &&
|
|
|
|
test_cmp list-expect list-actual
|
2020-01-31 21:16:08 +01:00
|
|
|
'
|
|
|
|
|
2020-02-11 16:02:24 +01:00
|
|
|
test_expect_success MINGW 'cone mode replaces backslashes with slashes' '
|
|
|
|
git -C repo sparse-checkout set deep\\deeper1 &&
|
|
|
|
cat >expect <<-\EOF &&
|
|
|
|
/*
|
|
|
|
!/*/
|
|
|
|
/deep/
|
|
|
|
!/deep/*/
|
|
|
|
/deep/deeper1/
|
|
|
|
EOF
|
|
|
|
test_cmp expect repo/.git/info/sparse-checkout &&
|
|
|
|
check_files repo a deep &&
|
|
|
|
check_files repo/deep a deeper1
|
|
|
|
'
|
|
|
|
|
2019-11-21 23:04:33 +01:00
|
|
|
test_done
|