Thread (14 messages) flat view 14 messages, 3 authors, 2016-06-15

Re: [PATCH 1/9] remote-bzr: trivial cleanups

From: Felipe Contreras <hidden>
Date: 2016-06-15 22:57:01

On Fri, Apr 26, 2013 at 2:30 PM, Ramkumar Ramachandra
[off-list ref] wrote:
[completely off-topic; don't worry, we're just having a friendly chat]

Felipe Contreras wrote:
quoted
If you are not prepared to defend your review, so are others, why to
you blame that on me? If you were right, you would be shown to be
right. Period.
Felipe, there are some things that are worth arguing about for a long
time (like the new revision spec I'm proposing in [1]), and others
that are not.

[1]: http://thread.gmane.org/gmane.comp.version-control.git/222248/focus=222526
I agree, and I have discussed about issues with diff in the past,
unfortunately didn't reach any conclusion. I've tried to follow that
thread, but I don't really know what is actually being proposed
anymore.

If you are so keen in receiving feedback from your fellow developers,
you should eventually send an email summarizing the issues and the
proposal for everyone to understand.

In contrast, I take it you agree this trivial patch is not worth
discussing, which I agreed in my first reply.
quoted
No, we operate in evidence and reason, *not* opinion. Any reasonable
person would say "well, I *think* this commit message needs more
description, but I don't *know*, I don't have *evidence* for it, so
I'm not going to fight to the death, as if I had".
Don't you think you're taking reason to an extreme here?  Reason is a
tool that I use when I want.  I don't want reason when I'm browsing
Google Art Project or listening to Gentle Giant.  Arguments like "is
this commit message large enough?" are really not worth the time and
effort.
Reason is not a tool for appreciating art, reason is a tool for
discovering truth, and if when arguing you are not interested in what
is actually true, I'm not interested in arguing with you.
quoted
Ultimately the decision to merge or not to merge comes to Junio, if
you don't like his decision, go complain to him, but I would be
prepared with points in time where people complained about these
patches, and there are no complains, so you have no ammunition at all
whatsoever.
I have no desire to "attack" you, Felipe.  I respect you as a more
experienced developer than myself, and am trying to offer constructive
criticism.

I don't have an ego (or consider myself important to the community).
Whatever will happen will happen, with or without me.
I appreciate your criticism, but that doesn't mean I must agree with
it. And if I do agree, that doesn't mean I must act upon it.
quoted
And this is how communities die. When everybody thinks the same, and
everyone who thinks differently is displaced. A monoculture, a place
full of yes-men where nobody criticizes anybody, a circlejerk where
everyone palms the back of everyone else. Eventually things go south,
and nobody around you understands why.

Diversity in a community is healthy. If you don't like people who
think differently, *you* have a problem. If you don't like standing up
and defending your ideas, *you* have a problem. If you don't like
discussing on the basis of evidence and reason, *you* have a problem.
Diversity is certainly healthy, and I it would be nice to have you in
the community.  We just have to find a way to keep the conflict down.
A fine way to start is to not rattle away in trivial inconsequential patches.

Cheers.

-- 
Felipe Contreras
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help