On Mon, Jun 10, 2013 at 06:58:47PM +0530, Ramkumar Ramachandra wrote:
> I've tried to write down a bare minimum, without restating the obvious.
I often come across so-called "community guidelines" in other
projects---some of which adhere to them quite strictly, and others simply
document something for the curious. But usually the reason for their
existence in the first place are tell-tale signs of trying to fix a problem
at the wrong end, and I believe this is what is about to happen if a
document such as this ever becomes official.
There's no disputing the fact that over the last few weeks, FC's behaviour
has been called in to question. He's managed to rub a lot of core people up
the wrong way, and in doing so has deliberately side-stepped that problem by
doing the one thing which puts anyone trying to raise that point muted; by
assuming that he's right.
It's a point on which one is never going to win, because no matter what one
says, it'll just get twisted round in such a way that one then ends up
questioning their own words, and their own conduct, and that's bad, because
there never was anything wrong with them to begin with.
So when you realise this point, it becomes almost impossible to proceed
further with any kind of discussion, because even the technical points of
discussion then end up being lost in a tirade of needless side-stepping
discussion. It's a bullying tactic on the part of FC which means he'll do
any, and everything, to get his own way.
So I say to all those seasoned reviewers out there on this list not to put
up with it.
There comes a point, regardless of how useful someone's contribution may be,
that if the barrier to entry is so high that any kind of criticism or
comment made against code comes with a massive chance of having to defend
yourself against innocence on the part of the reviewer, that those
contributions should be shelved. I've seen also another yardstick used to
defend FC's behaviour, and that is one of "commits within the last three
months". That count is completely meaningless, since the review process is
always going to be the same.
So these guidelines gain the community nothing, and only serve to punish
those who are already following them, without them being written down,
because the root-cause of the problem is still here, and isn't going to go
away, no matter how much referring to these guidelines might help.
That is why I think this is the wrong thing to do.
-- Thomas Adam
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html