Code review comment for lp:~amir-mohammadkhani/kicad/ash

Revision history for this message
Dick Hollenbeck (dickelbeck) wrote :

> And yes, I find it highly confrontational, borderline insulting if one of the highest ranking contributors to the project flat out denies the patch without studying it. This is not a way to attract new contributors. If you were some causal contributor it might be different, but the way you did it, putting all your weight behind it in the first post, shredding it can only mean you wanted to drown the patch as fast as possible and try to bully out someone using VS before he becomes a regular contributor.
>

I *did* want to drown the patch as a unit.

Some parts however I did commit, and had to do that by hand.

Perhaps I did let my frustration with Microsoft issues ruining the time
I spent on the CMakeList.txt editing affect my response. Those COMMENT
lines were put in there by me, at some investment of time to make them
look nice. Then here comes Microsoft crapping on me again, I go ballistic.

I am human:

1) I am permanently wired to resent anything that will cause *me* to do
anything to help Microsoft accomplish anything, including making you
happy with Visual C++. Changing the CMakeLists.txt file sacrifices my
investment in that effort. My time is money, and I see this as me
spending money helping Microsoft win. It ain't going to happen.

2) I resent it when folks waste my time, this is anybody, but especially
Microsoft. You have no idea what Microsoft has cost me over the years.

This is my lens of the issue.

I did not mean to insult you, and I apologize for not simply sending you
a more detailed explanation of my response. You are correct, it was not
particularly welcoming.

But again, if the Microsoft crap keeps falling on my head, I will have
to do something to eliminate that. Others may worship them, I avoid them.

Dick

« Back to merge proposal