Re: [PATCH v7 15/16] wt-status: expand added sparse directory entries
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Jul 12, 2021, 13:56 UTC
- Message-ID
- <3cf96b5f-3a8d-c00c-5d9e-c7095c058059@gmail.com>
- In-Reply-To
- <CABPp-BE=qw9pO89hggUFbu=eovfL=7Os5BY8DZNd5Z=qU==wFg@mail.gmail.com>
On 7/8/2021 9:03 PM, Elijah Newren wrote:
> On Mon, Jun 28, 2021 at 7:05 PM Derrick Stolee via GitGitGadget > <gitgitgadget@gmail.com> wrote:
...
Show 29 quoted lines
>> +test_expect_success 'reset mixed and checkout orphan' ' >> + init_repos && >> + >> + test_all_match git checkout rename-out-to-in && >> + >> + # Sparse checkouts do not agree with full checkouts about >> + # how to report a directory/file conflict during a reset. >> + # This command would fail with test_all_match because the >> + # full checkout reports "T folder1/0/1" while a sparse >> + # checkout reports "D folder1/0/1". This matches because >> + # the sparse checkouts skip "adding" the other side of >> + # the conflict. > > The same issue I highlighted last time is still present. If you > insert an "exit 1" right here, then run > ./t1092-sparse-checkout-compatibility.sh --ver --imm -x > until it stops, then > cd t/trash directory.t1092-sparse-checkout-compatibility/sparse-checkout > git ls-files -t | grep folder # Note the files that are sparse > git reset --mixed HEAD~1 > git ls-files -t | grep folder # Note the files that are sparse -- > there are some that aren't that should be > git sparse-checkout reapply > git ls-files -t | grep folder # Note the files that are sparse > > Granted, this is a bug with sparse-checkout without sparse-index, so > not something new to your series. But since you are using comparisons > between regular sparse-checkouts and sparse-index to verify > correctness, this seems problematic to me.
I'll add it to the pile, but I want to continue having this series focus on making the sparse-index work quickly without a change in behavior from a normal index. Changing the behavior of the sparse- checkout feature should be a separate series.
Thanks, -Stolee