<div dir="ltr">On 13 November 2013 11:01, Frank Henigman <span dir="ltr"><<a href="mailto:fjhenigman@google.com" target="_blank">fjhenigman@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I'd like to see an explanation of "signed-off-by," "reviewed-by" etc.  Maybe as simple as:<div>
<br></div><div>"Use the tags signed-off-by, reviewed-by, tested-by, acked-by as for linux kernel patches</div>
<div>(see <a href="https://www.kernel.org/doc/Documentation/SubmittingPatches)" target="_blank">https://www.kernel.org/doc/Documentation/SubmittingPatches)</a>."</div></div></blockquote><div><br></div><div>That seems reasonable.  Note, however, that Piglit doesn't consistently use "signed-off-by":<br>
<br>$ git log master | grep '^commit' | wc<br>   4885    9770  234480<br>$ git log master | grep -i 'signed-off-by' | wc <br>   1241    4969   70925<br><br></div><div>(If you'd like to encourage us to start using "signed-off-by" consistently, I'm happy to have a policy discussion about that, but the discussion should happen in its own email thread rather than here, so that more people will see it).<br>
<br></div><div>I'll follow up with a patch taking your suggestion into account.<br></div></div></div></div>