<p> Attention is currently required from: Kevin Harwell. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17334">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17334?tab=comments">Patch Set #2:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see the harm in minor breakages to that opinion.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">You gave a minus one. You gave a direct order. Consequently, the change is somehow blocked for a valid reason. I have then to resolve that either by arguing or following the order. There was once a Wiki that stated, as a contributor, I am not allowed to argue. I do not find that any more. Anyway, that behavior further burdens me because I have to find a way to get rid of that -1, although my change was totally valid. Even checking (and stating) whether I am unwilling or unable takes additional effort to come to that conclusion. Especially ‘unable’ is problematic in public. Even for me, after so many commits, changing a commit in review is a daunting task (perhaps I still misuse Gerrit). Others might not be able at all. Especially when the change is not a simple cherry-pick over all branches but different in some branches. Therefore, even such minor breaks can have a disastrous outcome. In any case, it makes it more complicated for me, someone who might not benefit from the change in the first place.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Finally, you cannot check whether the other one understood and double-checked your order, or blindly followed it. Consequently, you have the problem of a transitive +1.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Another example: With my very first commit in the Asterisk project, I got a -1 because I copied a log message statement from code around me. That did not use ast_verbose, ast_debug, ast_log. At that time, I did not even know about those symbols, I simply copied over the behavior around that code. I then had to learn and understand why I got a -1. Even that I did not get. And then learn that logger stuff. And then I had even to change another code line because I touched it semantically (the indention of that line of code changed because of my change). All this increases the burden on the contributor.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Consequently, I suggest staying away even from minor breakages. If you want to avoid an additional change on your behalf, why not just suggest it (not order it) and give a +/-0 (not a -1), so original change is not blocked, and the contributor is not *obligated* to respond.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Another example: I found this particular group-command issue in 650 projects in Debian. In 650. If 1 of 100 take minor breakage, this turns into around the effort same as 657 projects. No problem. If 10 of 100 take minor breakage, this gets roughly the effort of 715 projects. Perhaps I have simply not the time even explain to 65 projects, I am not willing to do that. In other words, what you think is minor, might accumulate to a really big one for the other party.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17334">change 17334</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/17334"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I4687857b9d56e6f44fd440b73af156691660202e </div>
<div style="display:none"> Gerrit-Change-Number: 17334 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 30 Nov 2021 10:22:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Comment-In-Reply-To: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Comment-In-Reply-To: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>