[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