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

Paul Walmsley paul at pwsan.com
Wed May 28 18:48:47 UTC 2014


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:".


- Paul


More information about the Ksummit-discuss mailing list