On Mon, Sep 09, 2019 at 01:08:24PM -0400, Jeff King wrote: > I'll work up what I sent earlier into a real patch, and include some of > this discussion. Here it is. I pulled Jon's tests out into their own patch (mostly because it makes it easier to give credit). Then patch 2 is my fix, and patch 3 is the message fixups he had done. This replaces what's queued in js/partial-clone-sparse-blob. [1/3]: t5616: test cloning/fetching with sparse:oid=<oid> filter [2/3]: list-objects-filter: delay parsing of sparse oid [3/3]: list-objects-filter: give a more specific error sparse parsing error builtin/rev-list.c | 4 ---- list-objects-filter-options.c | 14 ++------------ list-objects-filter-options.h | 2 +- list-objects-filter.c | 14 +++++++++++--- t/t5616-partial-clone.sh | 36 +++++++++++++++++++++++++++++++++++ 5 files changed, 50 insertions(+), 20 deletions(-)