Re: [PATCH 1/1] [gdb]: add git trailer information on gdb/MAINTAINERS

Simon Marchi via Gdb
2023-05-17T14:35:33Z

> My reading of the kernel's documentation of the tag[1] makes it sound like it is a partial approval, especially the lines:
>
> |
>
> Acked-by: is not as formal as Signed-off-by:. It is a record that the acker
> has at least reviewed the patch and has indicated acceptance.|
>
> And the only time I saw it used in QEMU seems to corroborate that reading. That said, I can be misunderstanding here (english is not my native language after all).
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=HEAD#n441

Thanks for the pointers. It looks like Acked-By can fit a range of
situations, as long as it fits in "I didn't look at all of it or didn't
do a thoroug review, but what I have seen LGTM". I think it fits my use
case of just reading the commit message and agreeing with that.

Simon