This was noticed at $DAYJOB when in a partial clone with a filter other than blob:none. The 1st patch is a code cleanup (so that the 2nd patch does not need to take into account the unused variable), and the 2nd patch contains the bug fix. Jonathan Tan (2): promisor-remote: remove unused variable fetch: do not override partial clone filter builtin/fetch.c | 2 +- list-objects-filter-options.c | 10 +++++++++- promisor-remote.c | 5 ----- promisor-remote.h | 2 +- t/t5601-clone.sh | 5 +++++ 5 files changed, 16 insertions(+), 8 deletions(-) -- 2.28.0.681.g6f77f65b4e-goog
