<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:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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.<br>The<br>reason for your -1 got void. In such a case, you remove that -1.<br>If<br>you have a new or different concern, you add that with a new -1.<br>I<br>cannot see in your mind and know whether you got it or whether<br>you<br>have some different reason. I was stranded, because I did not<br>know<br>how to proceed.<br>> 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<br>then<br>rebase my change because it revealed something else. I am unhappy<br>about that rebase in particular because that really takes my<br>time.<br>By the way, I have to create a complete new commit message. You<br>are<br>blocking mine previous legitimate change. Furthermore, instead of<br>getting a ‘thank you’ or at least ‘please’, you order me to go<br>for<br>your suggestion like being a subordinate of yours.<br>> Consequently, I have to align with a new situation and again find<br>a<br>solution to get this issue solved finally. This is punishment of<br>external contributors and unacceptable behavior in a code review.<br>I<br>love to help and sometimes one unveils bigger things. OK.<br>However,<br>the one doing all this should not be placed even higher burdens<br>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><p style="white-space: pre-wrap; word-wrap: break-word;">I don’t think that anything Corey was saying was meant to be<br>malicious in nature.  As developers, respectful language is<br>important, but I think as developers in a peer review process we<br>try to be technical first and sometimes forget to be personable as<br>well.  I didn’t see anything overtly unkind on the comments of this<br>review, so perhaps there’s maybe a language barrier mismatch in<br>terms of gerrit/text communication.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As I’m sure you know, just because we contribute a patch, it<br>doesn’t necessarily mean that it will automatically be included.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Corey had at least one request you didn’t address originally - that<br>is your choice as someone contributing code, but in a peer reviewed<br>code contribution process, the purpose is to make sure that you<br>have technical buy-in from other interested and vested parties that<br>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<br>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<br>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<br>one of the more regular contributors to the project and I think<br>that everybody is grateful for that.  Part of the responsibility of<br>working on an open source project with lots of stakeholders is to<br>make sure your patch passes the review process - that is what keeps<br>the code clean and how we try to catch bugs or problems before they<br>are included.  That can be a frustrating process when you’d like to<br>get your patch in quickly, but in order for this particular patch<br>to move forward, the suggested changes need to be included (unless<br>there’s something specifically wrong with them).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Best wishes,<br>Matthew Fredrickson</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Somehow the formatting got messed up in my response.  It was meant to be all plaintext, but for some reason the first two paragraphs got bungled.</p><p style="white-space: pre-wrap; word-wrap: break-word;">They weren't meant to have any special emphasis or treatment in my response, FYI :-)</p><p style="white-space: pre-wrap; word-wrap: break-word;">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:29:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>