Re: acked-by meaning

["Rafael J. Wysocki" <rjw@sisk.pl>]

On Tuesday, November 13, 2012 02:12:19 PM Frank Rowand wrote:
> On 11/11/12 19:47, Randy Dunlap wrote:
> > On 11/11/2012 05:40 PM, Thiago Farina wrote:
> > 
> >> Hi,
> >>
> >> What is the meaning of 'Acked-by:' line? Is the same of LGTM? How it
> >> differs from 'Reviewed-by:'?
> > 
> 
> > Reviewed-by: is stronger than Acked-by:.
> > If someone replies with Reviewed-by, they also accept some
> > responsibility for fixing any problems that the patch might
> > introduce after it is merged.
> 
> I haven't found a "responsibility for fixing any problems" in the
> references provided below.  Can you help me out by pointing
> to a specific sentence or another reference?

"Reviewed-by" informs the maintainer of the relevant subsystem that the
reviewer has spent time to go through the patch and understand what it
does in detail.  It also implies that the reviewer has no objections against
the patch.

The maintainer can use this information when deciding whether or not to
apply the patch, possibly without spending as much time on it as would be
necessary otherwise.

> > Acked-by just means agreement with the patch.

Actually, the meaning of "Acked-by" depends on who gives it.

If the patch touches multiple subsystems and their maintainers "ack" it,
they inform each other that they have no objections against it and, from their
perspective, it can be merged if any one of them wants to take it.  So in that
particular case it's rather strong.

On the other hand, if the "ack" is given by someone who's not a maintainer,
the meaning of it is pretty much the same as for a "like" on Facebook. :-)

Thanks,
Rafael


-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.