On Mon, Jun 10, 2013 at 5:55 PM, Phil Hord <phil.hord@xxxxxxxxx> wrote: > On Sun, Jun 9, 2013 at 3:37 PM, Felipe Contreras > <felipe.contreras@xxxxxxxxx> wrote: >> >> On Sun, Jun 9, 2013 at 2:24 PM, Felipe Contreras >> <felipe.contreras@xxxxxxxxx> wrote: >> > Same as before, but: >> >> Also, remove the patches from Martin von Zweigbergk, because >> apparently some people have trouble understanding that they were not >> part of this series. > > Please try not to sound disgruntled. This attitude is toxic. You have > turned this change into a complaint: that "some people have trouble > understanding" which shows a genuine lack of understanding and > compassion on your part. Instead you can phrase your change notes > more helpfully if you make changes only when you yourself actually > believe the change should be made. If you cannot do this, perhaps you > can pretend. That would be dishonest. Moreover, there wasn't a good reason to remove these patches, I made it clear I added those patches only to make sure the real patches of this series worked correctly. Also, I clarified that to Thomas Rast[1], only to receive a totally unconstructive comment[2]. Why don't you ask Thomas Rast to be more constructive[2]? Then Johan Herland uses that as an example of a constructive comment[3]. Why don't you correct Johan Herland? No, you pick the easy target: me. I already dd more than my fair share by carrying these 36 patches through several iterations, yet you ask *more* of me. Why don't you ask more of the people that just hit reply on their MUA? Thomas' task was easy; he simply had to say "Oh, these aren't meant to be applied, got it." [1] http://article.gmane.org/gmane.comp.version-control.git/227039 [2] http://article.gmane.org/gmane.comp.version-control.git/227040 [3] http://article.gmane.org/gmane.comp.version-control.git/227102 -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html