Re: [PATCH v5 0/9] Sparse index: delete ignored files outside sparse cone
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Sep 8, 2021, 11:39 UTC
- Message-ID
- <4a0125d2-67d9-b533-4f8b-618c3de3b411@gmail.com>
- In-Reply-To
- <xmqqczpj7dyk.fsf@gitster.g>
On 9/8/2021 2:56 AM, Junio C Hamano wrote:
Show 27 quoted lines
> Junio C Hamano <gitster@pobox.com> writes: > >> "Derrick Stolee via GitGitGadget" <gitgitgadget@gmail.com> writes: >> >>> Updates in V5 >>> ============= >>> >>> * Updated the locality of a cache_entry pointer. >>> >>> * Rephrased a comment. >>> >>> * Removed the patch adding a config option. >>> >>> * I tried, but failed, to create a scenario where the call to >>> cache_tree_update() causes a test to fail. I still think this is valuable >>> as defensive programming for the reasons mentioned in the patch, which is >>> why I didn't remove them here. >> >> OK. >> >> The 'sg/test-split-index-fix' topic that has been queued in 'seen' >> textually depended on the earlier iteration of this topic, so I'll >> discard it while replacing this topic. > > I rebased the dependent topic myself, but I would not be surprised > if I weren't careful enough and introduced a new bug. Please double > check when a new 'seen' is pushed out in several hours.
Did you have any conflicts? I don't think this version changed any of the textual dependencies. Anyway, the test suite passes with GIT_TEST_SPARSE_INDEX=1 on my machine at 'seen' (except for an issue with t7900-maintenance.sh that is unrelated).
Thanks, -Stolee