<div dir="ltr">So my interpretation of Ben's response is:<br><div><br></div><div>Acked-by a human ~= +1<br></div><div>Reviewed-by ~= +2<br></div><div>Tested-by ~= +3</div><div><br></div><div>And the convention is we wait for a +2 to merge.<br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">David E DeMarle<br>Kitware, Inc.<br>R&D Engineer<br>21 Corporate Drive<br>Clifton Park, NY 12065-8662<br>Phone: 518-881-4909</div></div>
<br><div class="gmail_quote">On Wed, Mar 25, 2015 at 4:41 PM, Ben Boeckel <span dir="ltr"><<a href="mailto:ben.boeckel@kitware.com" target="_blank">ben.boeckel@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Mar 25, 2015 at 15:23:15 -0400, Sean McBride wrote:<br>
> The "Develop VTK with Git" page does not make clear to me what the<br>
> equivalent of +2 is now...<br>
><br>
> - What's the difference between someone adding Acked-by vs Reviewed-by<br>
>   vs Signed-off-by vs Tested-by?<br>
<br>
</span>My interpretations:<br>
<br>
Acked-by    - "I see nothing wrong with the change" (which is all<br>
              kwrobot's checks can do)<br>
Reviewed-by - "I reviewed the change and agree with it" (we should<br>
              probably use this rather than Acked-by for humans)<br>
Tested-by   - "I tested the change and approve that it works" (buildbot<br>
              will be using this along with Rejected-by and Acked-by)<br>
<br>
Signed-off-by is typically used to signify "I understand that my<br>
contribution is made under the license of the project and my patches are<br>
licensed accordingly" and is for the submitter to add when making<br>
commits and sending them to a list via git's format-patch / send-email<br>
workflow. The act of submitting a merge request is taken as this<br>
permission here. The manpage for git's format.signoff configuration:<br>
<br>
    format.signoff<br>
        A boolean value which lets you enable the -s/--signoff option of<br>
        format-patch by default.  Note: Adding the Signed-off-by: line<br>
        to a patch should be a conscious act and means that you certify<br>
        you have the rights to submit this work under the same open<br>
        source license. Please see the SubmittingPatches document for<br>
        further discussion.<br>
<span class=""><br>
> - Which must be present for it to be considered agreed upon?<br>
> - When do I know I can merge a change in?<br>
<br>
</span>The only hard gate right now is that kwrobot has OK'd the change. There<br>
is a patch for kwrobot I need to finish to add in support for blocking<br>
based on a developer's "Rejected-by" trailer and requiring at least one<br>
Acked-by (or, better, Reviewed-by) from a developer that is not the<br>
submitter.<br>
<span class="HOEnZb"><font color="#888888"><br>
--Ben<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br></div>