[Asterisk-code-review] BuildSystem: Go for Python 2 explicitly, because that is cur... (asterisk[master])

Alexander Traud asteriskteam at digium.com
Thu Apr 19 03:09:04 CDT 2018


Alexander Traud has posted comments on this change. ( https://gerrit.asterisk.org/8673 )

Change subject: BuildSystem: Go for Python 2 explicitly, because that is currently needed.
......................................................................


Patch Set 1:

Glad you got it finally. However, the behavior is unacceptable. The reason for your -1 got void. In such a case, you remove that -1. If you have a new or different concern, you add that with a new -1. I cannot see in your mind and know whether you got it or whether you have some different reason. I was stranded, because I did not know how to proceed.

OK, you gave another -1: ‘Please, wait for the other change’. Again, this is not acceptable behavior, because I am punished for revealing all this stuff: I have to argue, convince, wait, and then rebase my change because it revealed something else. I am unhappy about that rebase in particular because that really takes my time. By the way, I have to create a complete new commit message. You are blocking mine previous legitimate change. Furthermore, instead of getting a ‘thank you’ or at least ‘please’, you order me to go for your suggestion like being a subordinate of yours.

Consequently, I have to align with a new situation and again find a solution to get this issue solved finally. This is punishment of external contributors and unacceptable behavior in a code review. I love to help and sometimes one unveils bigger things. OK. However, the one doing all this should not be placed even higher burdens on his shoulder. If you punish the one who reveals something, he is not doing this again.


-- 
To view, visit https://gerrit.asterisk.org/8673
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I29e694dc7e92510dc26b15895b55f78f67146b2c
Gerrit-Change-Number: 8673
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Comment-Date: Thu, 19 Apr 2018 08:09:04 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180419/67cb0d44/attachment.html>


More information about the asterisk-code-review mailing list