Elijah Newren <newren@xxxxxxxxx> writes: > On Wed, Dec 22, 2021 at 6:20 AM Derrick Stolee via GitGitGadget > <gitgitgadget@xxxxxxxxx> wrote: >> >> This is now based on 'master'. >> >> Did you know that 'fetch' and 'pull' read the index? I didn't, or this would >> have been an integration much earlier in the cycle. They read the index to >> look for the .gitmodules file in case there are submodules that need to be >> fetched. Since looking for a file by name is already protected, we only need >> to disable 'command_requires_full_index' and we are done. >> >> The 'ls-files' builtin is useful when debugging the index, and some scripts >> use it, too. We are not changing the default behavior which expands a sparse >> index in order to show all of the cached blobs. Instead, we add a '--sparse' >> option that allows us to see the sparse directory entries upon request. > ... >> Updates in v2 >> ============= >> >> * Rebased onto latest ld/sparse-index-blame without issue. >> * Updated the test to use diff-of-diffs instead of a sequence of greps. >> * Added patches that remove the use of 'test-tool read-cache --table' and >> its implementation. >> >> >> Updates in v3 >> ============= >> >> * Fixed typo in commit message. >> * Added comments around doing strange things in an ls-files test. >> * Fixed adjacent typo in a test comment. >> >> >> Updates in v4 >> ============= >> >> * Rebased on to 'master' now that ld/sparse-index-blame is merged. >> * Change testing strategy to check exact output instead of using 'diff -u'. >> * Updated documentation to state that directories have a trailing slash. > > This version looks good to me: > > Reviewed-by: Elijah Newren <newren@xxxxxxxxx> Yup, they looked good to me too. Thanks.
