Re: [PATCH 2/3] docs: submitting-patches: clarify difference between Acked-by and Reviewed-by

From: Jonathan Corbet
Date: Sun Jan 12 2025 - 14:59:39 EST


Neal Gompa <neal@xxxxxxxxx> writes:

> I've had my Reviewed-by tags silently ignored or deliberately stripped
> because even though I've done a technical review, the maintainer does
> not believe that I did. Therefore, what I am saying is that
> maintainers seem to speciously decide whether an Acked-by or
> Reviewed-by tag is appropriate or not *after* someone has sent it.
>
> This is the fundamental problem I have right now. This decision is not
> the maintainer's to make, it is the submitter's.

There *are* people who seem to make a game of getting as many such tags
into the repository as possible. I think a bit of maintainer discretion
is important here; I don't believe that there is a fundamental right to
inject a tag into somebody else's patch...?

jon