Re: [PATCH v2] log: add option to search for header or body

From: Phillip Wood <phillip.wood123@gmail.com>
Date: Apr 7, 2024, 14:00 UTC
Message-ID: <0bad6e79-42c6-4149-92f0-58c4726080b1@gmail.com>
In-Reply-To: <xmqq4jcdhfht.fsf@gitster.g>

On 07/04/2024 07:08, Junio C Hamano wrote:

Show 15 quoted lines

> "Max Coplan via GitGitGadget" <gitgitgadget@gmail.com> writes:
> I am imagining a pair of new options to name people (all OR'ed) and
> to name places the names of these people should appear (again, all
> OR'ed).  I am not good at naming, so the option names in the example
> is not more than illustration of an idea and not my recommendation,
> but a command:
> 
>      git log --by="Max Coplan" --by="Junio C Hamano" \
> 	    --by-where=author,Signed-off-by,Co-authored-by
> 
> would find a commit that has one (or more) of the given names
> in one (or more) of the places that are specified, where the places
> can be either "author", "committer" to specify these headers in the
> commit object, or random other string to specify trailer lines with
> given keys.

I like this, yesterday[1] I didn't have a clear idea in my head of how such an option should work but I think passing the names and the fields to match those names against as two separate options is a good idea.

Best Wishes

Phillip

[1] https://lore.kernel.org/git/c93817ba-5945-4ec0-9775-5621481b972c@gmail.com/