diff options
| author | Derrick Stolee <dstolee@microsoft.com> | 2020-01-31 20:16:15 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2020-01-31 21:05:29 (GMT) |
| commit | f998a3f1e588d73ed7285cb14ac4839f63f6dc82 (patch) | |
| tree | 264439faaceedd68588b444cd191b450fbd1e700 /t/t1091-sparse-checkout-builtin.sh | |
| parent | d2e65f4c9056be72ff8a1f39245c5e1b27d556b2 (diff) | |
| download | git-f998a3f1e588d73ed7285cb14ac4839f63f6dc82.zip git-f998a3f1e588d73ed7285cb14ac4839f63f6dc82.tar.gz git-f998a3f1e588d73ed7285cb14ac4839f63f6dc82.tar.bz2 | |
sparse-checkout: fix cone mode behavior mismatch
The intention of the special "cone mode" in the sparse-checkout
feature is to always match the same patterns that are matched by the
same sparse-checkout file as when cone mode is disabled.
When a file path is given to "git sparse-checkout set" in cone mode,
then the cone mode improperly matches the file as a recursive path.
When setting the skip-worktree bits, files were not expecting the
MATCHED_RECURSIVE response, and hence these were left out of the
matched cone.
Fix this bug by checking for MATCHED_RECURSIVE in addition to MATCHED
and add a test that prevents regression.
Reported-by: Finn Bryant <finnbryant@gmail.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t1091-sparse-checkout-builtin.sh')
| -rwxr-xr-x | t/t1091-sparse-checkout-builtin.sh | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/t/t1091-sparse-checkout-builtin.sh b/t/t1091-sparse-checkout-builtin.sh index 37e9304..7d98209 100755 --- a/t/t1091-sparse-checkout-builtin.sh +++ b/t/t1091-sparse-checkout-builtin.sh @@ -305,6 +305,18 @@ test_expect_success 'different sparse-checkouts with worktrees' ' check_files worktree a deep ' +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 +' + check_read_tree_errors () { REPO=$1 FILES=$2 |