Re: [RFC 0/2] explicitly support or not support --exclude-promisor-objects

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 23, 2018, 04:48 UTC
Message-ID: <xmqqmur5s1y9.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <cover.1540256910.git.matvore@google.com>

Matthew DeVore <matvore@google.com> writes:

Show 5 quoted lines

> This patch set fixes incorrect parsing of the --exclude-promisor-objects
> option that I found while working on:
>
>   https://public-inbox.org/git/cover.1539298957.git.matvore@google.com/
>

Thanks; both patches make sense.

As the problematic feature appeared in 2.17.x track, I'll see if I can easily make it ready to be merged down to maint-2.17 track later when somebody wants to.

Show 13 quoted lines

> Matthew DeVore (2):
>   Documentation/git-log.txt: do not show --exclude-promisor-objects
>   exclude-promisor-objects: declare when option is allowed
>
>  Documentation/rev-list-options.txt | 2 +-
>  builtin/pack-objects.c             | 1 +
>  builtin/prune.c                    | 1 +
>  builtin/rev-list.c                 | 1 +
>  revision.c                         | 3 ++-
>  revision.h                         | 1 +
>  t/t4202-log.sh                     | 4 ++++
>  t/t8002-blame.sh                   | 4 ++++
>  8 files changed, 15 insertions(+), 2 deletions(-)