[Ksummit-discuss] Reforming Acked-by (was Re: [TOPIC] Encouraging more reviewers)

H. Peter Anvin hpa at zytor.com
Thu May 29 15:58:42 UTC 2014


On 05/28/2014 11:48 AM, Paul Walmsley wrote:
> 
> Also long-overdue is a clarification on exactly what "Acked-by" means.
> Right now it is being used for at least two distinct and
> mutually-incompatible purposes:
> 
> 1. A maintainer A for code affected by a patch, who is distinct from a
> maintainer B queuing a patch, has reviewed the patch and has cleared it as
> being OK for maintainer B to send upstream
> 
> 2. A casual review has been done by someone who is not a maintainer for
> the code in question
> 
> What I would propose is to have the first use replaced by a new tag, 
> "Maintainer-acked-by:", and the second use abolished, along with 
> "Acked-by:", and replaced by "Reviewed-by:".
> 

The distinction is on the right side of the colon.  It's a matter of
reputation.

	-hpa




More information about the Ksummit-discuss mailing list