<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 3, 2014 at 8:46 AM, Vadim Bendebury <span dir="ltr"><<a href="mailto:vbendeb@chromium.org" target="_blank">vbendeb@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Note that the version of gerrit used for chromeos allows editing the<br>

commit message through the web interface, no need to re-upload the<br>
patch, and the scores are kept.<br>
<br>
Not sure if this is available in the coreboot gerrit though.<br></blockquote><div><br></div><div>Yes, it is. I happen to think that is sufficient to accomplish what Paul is asking w.r.t. fixing commit messages.</div><div>
<br></div><div>But I also like the idea of keeping scores for trivial rebases and no code change as well. There's no sense in having to wait for somebody to re-approve a patch for such things.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
--vb<br>
<div class=""><div class="h5"><br>
<br>
On Wed, Sep 3, 2014 at 12:39 AM, Paul Menzel<br>
<<a href="mailto:paulepanter@users.sourceforge.net">paulepanter@users.sourceforge.net</a>> wrote:<br>
> Dear coreboot folks,<br>
><br>
><br>
> on IRC I was told, developers won’t update or fix commit messages as<br>
> long as it means +2 scores are removed.<br>
><br>
> Luckily Gerrit supports that [2].<br>
><br>
>         A useful feature on labels is the possibility to automatically<br>
>         copy scores forward to new patch sets if it was a trivial rebase<br>
>         or if there was no code change (e.g. only the commit message was<br>
>         edited).<br>
><br>
> I propose to enable these two features.<br>
><br>
> If nobody opposes until next Monday, September 8th, it’d be great if the<br>
> two config parameters below could be set to true.<br>
><br>
> 1. label.Label-Name.copyAllScoresOnTrivialRebase [3]<br>
> 2. label.Label-Name.copyAllScoresIfNoCodeChange [4]<br>
><br>
><br>
> Thanks,<br>
><br>
> Paul<br>
><br>
><br>
> [1] <a href="http://review.coreboot.org/Documentation/intro-project-owner.html#labels" target="_blank">http://review.coreboot.org/Documentation/intro-project-owner.html#labels</a><br>
> [2] <a href="http://review.coreboot.org/Documentation/intro-project-owner.html#labels" target="_blank">http://review.coreboot.org/Documentation/intro-project-owner.html#labels</a><br>
> [3] <a href="http://review.coreboot.org/Documentation/config-labels.html#label_copyAllScoresOnTrivialRebase" target="_blank">http://review.coreboot.org/Documentation/config-labels.html#label_copyAllScoresOnTrivialRebase</a><br>

> [4] <a href="http://review.coreboot.org/Documentation/config-labels.html#label_copyAllScoresIfNoCodeChange" target="_blank">http://review.coreboot.org/Documentation/config-labels.html#label_copyAllScoresIfNoCodeChange</a><br>

><br>
</div></div><span class=""><font color="#888888">> --<br>
> coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
> <a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br>
<br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a></font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>David Hendricks (dhendrix)<br>
Systems Software Engineer, Google Inc.
</div></div>