<p>Matthew Fredrickson <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8673">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Glad you got it finally. However, the behavior is unacceptable. The<br>reason for your -1 got void. In such a case, you remove that -1. If<br>you have a new or different concern, you add that with a new -1. I<br>cannot see in your mind and know whether you got it or whether you<br>have some different reason. I was stranded, because I did not know<br>how to proceed.</p><p style="white-space: pre-wrap; word-wrap: break-word;">OK, you gave another -1: ‘Please, wait for the other change’.<br>Again, this is not acceptable behavior, because I am punished for<br>revealing all this stuff: I have to argue, convince, wait, and then<br>rebase my change because it revealed something else. I am unhappy<br>about that rebase in particular because that really takes my time.<br>By the way, I have to create a complete new commit message. You are<br>blocking mine previous legitimate change. Furthermore, instead of<br>getting a ‘thank you’ or at least ‘please’, you order me to go for<br>your suggestion like being a subordinate of yours.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Consequently, I have to align with a new situation and again find a<br>solution to get this issue solved finally. This is punishment of<br>external contributors and unacceptable behavior in a code review. I<br>love to help and sometimes one unveils bigger things. OK. However,<br>the one doing all this should not be placed even higher burdens on<br>his shoulder. If you punish the one who reveals something, he is<br>not doing this again.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hey Alex,</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">I don’t think that anything Corey was saying was meant to be malicious in nature.  As developers, respectful language is important, but I think as developers in a peer review process we try to be technical first and sometimes forget to be personable as well.  I didn’t see anything overtly unkind on the comments of this review, so perhaps there’s maybe a language barrier mismatch in terms of gerrit/text communication.<br> <br>As I’m sure you know, just because we contribute a patch, it doesn’t necessarily mean that it will automatically be included.</pre><p style="white-space: pre-wrap; word-wrap: break-word;">Corey had at least one request you didn’t address originally - that is your choice as someone contributing code, but in a peer reviewed code contribution process, the purpose is to make sure that you have technical buy-in from other interested and vested parties that may be reviewing your code before the code is merged.</p><p style="white-space: pre-wrap; word-wrap: break-word;">After discussing this with Corey, I think the preference he and I landed on for the configure.ac changes is:</p><p style="white-space: pre-wrap; word-wrap: break-word;">AC_PATH_PROGS([PYTHON], [python python3 python2 python3.6 python2.7], :)</p><p style="white-space: pre-wrap; word-wrap: break-word;">and</p><p style="white-space: pre-wrap; word-wrap: break-word;">AST_PKG_CONFIG_CHECK([PYTHONDEV], [python < 3])<br>AST_PKG_CONFIG_CHECK([PYTHONDEV], [python2])<br>AST_PKG_CONFIG_CHECK([PYTHONDEV], [python-2.7])</p><p style="white-space: pre-wrap; word-wrap: break-word;">Alexander, as a project, we appreciate your contributions - you are one of the more regular contributors to the project and I think that everybody is grateful for that.  Part of the responsibility of working on an open source project with lots of stakeholders is to make sure your patch passes the review process - that is what keeps the code clean and how we try to catch bugs or problems before they are included.  That can be a frustrating process when you’d like to get your patch in quickly, but in order for this particular patch to move forward, the suggested changes need to be included (unless there’s something specifically wrong with them).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Best wishes,<br>Matthew Fredrickson</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8673">change 8673</a>. To unsubscribe, 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/8673"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I29e694dc7e92510dc26b15895b55f78f67146b2c </div>
<div style="display:none"> Gerrit-Change-Number: 8673 </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: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 20 Apr 2018 20:26:04 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>