Re: [PATCH v4 00/13] New remote-hg helper

On Wed, Oct 31, 2012 at 4:55 PM, Michael J Gruber
[off-list ref] wrote:

Felipe Contreras venit, vidit, dixit 31.10.2012 16:39:

quoted

This is precisely ad hominem; you are ignoring the code, not because
of the code, because of the person. This is as ad hominem as it gets.

I am not rejecting your code (I reviewed an early series) but reject the
communication style and manners displayed in this thread.

All right, you are not rejecting it, but you are staying away from it,
and presumably if it was coming from somebody else, you wouldn't.

quoted

As for how "professional or helpful" that is, it's debatable. The
Linux kernel mailing list is known for being harsh, and yet, they
manage to get more things done than any other. They truly look at the
code, just the code, they don't consider criticism to the code
personally (nobody should), nor linger on any personal beefs that only
distract from the end goal.

There are people who choose not to be on that list because of its style.
For this list, I think we should follow this list's style, not that one.

And what is this lists' style? I don't see any guidelines anywhere.

But my point wasn't that we should follow Linux's style, my point is
that it's debatable how one should engage in discussions.

And yet, I haven't seen where exactly did I throw those ad hominem
attacks. I can point you to where Johannes threw such attacks (or at
least snarky), to me, but I don't think that's relevant.

quoted

But enough about Johannes, if I go on to Max's branch and give a try
to the code, make a list of issues, run my extensive tests and so on,
and make a report of the status, and a comparison with my code. Would
that make it more likely for you to stop being a by-stander?

Sure, that's what I and others have asked for.

Except nobody ever provided a link to the actual patches. You are the
first one to do so.

quoted

You accused me of ad hominem, now I ask you; can you ignore any
personal biases and look at the code, and only at the code?

My efforts here prove that I either have no biases or ignore them. I'm
not going to ignore the style of communication, though.

And yet earlier before you said in this list "we prefer to discuss the
code, just the code", and now you are saying you are not going to
ignore the style of communication, which is not code, and yet you are
discussing about it.

As a patch
submitter, you ("generic you") want the attention of others as
reviewers. It's in your own (again "generic you") interest not to put
them off, in the same way as it's up to the submitter to argue why a
patch is desirable and correct.

Ah, so you are making me a favor by reviewing the code?

How about we concentrate on what's good for the project? Our users
don't care about petty personal beefs. Support to pull and push
mercurial repositories, _that_ they do care about.

Cheers.

--
Felipe Contreras