Private vs public [LWN.net]

Private vs public

Posted Oct 10, 2015 14:06 UTC (Sat) by nevets (subscriber, #11875)
In reply to: Private vs public by marcH
Parent article: Sharp: Closing a door

Actually, there is a lot of criticism in the private. We just don't see it because, well, it's private.

But private criticism may not work as well either. When you are in a private forum, you can make claims that just wont hold up water. But doing that in a public setting has a bigger impact and both sides really need to have the facts straight before they post. Otherwise, if you yell at someone and are incorrect yourself, you can have others yell at you. This has happened, and kept the one that was yelling in check.

Escalating to public shaming really should be the last resort, but should still be on the table. Everyone that posts code to the Linux kernel should be a little nervous about it. That nervousness keeps one in check. Once you can post code without any anxiety then you can easily post low quality code. I've been submitting code to the Linux kernel for over 10 years, and I'm still nervous with every patch I send out. That drives me to make sure my patch can stand on its own, and keeps me from doing something stupid. If you can't handle that, and want to be able to send out "no worries" patches, where if you totally screw up, nobody will criticize you, I'm sorry, you are not fit to submit to a project as serious as the Linux kernel.

to post comments

Private vs public

Posted Oct 10, 2015 14:09 UTC (Sat) by nevets (subscriber, #11875) [Link] (2 responses)

BTW, when I said "you" at the end, I don't mean you :-) That was just a generalization. I like German because it has a separate word for that "man", which doesn't mean the English "man".

Private vs public

Posted Oct 10, 2015 15:51 UTC (Sat) by cebewee (guest, #94775) [Link]

I should be able to use "one" in (most of) those case.

Private vs public

Posted Oct 15, 2015 8:02 UTC (Thu) by paulj (subscriber, #341) [Link]

You can use "one" as a general abstract pronoun for people, as a contraction of 'someone', in conjunction with "they", "them" to reference that abstract person, etc., e.g.:

"If someone can't handle that, and they want to be able to send out "no worries" patches, where if they totally screw up, nobody will criticize them, I'm sorry, they are not fit to submit to a project as serious as the Linux kernel."

"one" just happened not to fit easily in there, but often used in "One might think", "If one wants to avoid criticism". Note that "one" can come across as a bit stilted, just using "someone" usually sounds better IMO. "someone" is pretty much the same as the germanic "jemand" / "iemand".

Private vs public

Posted Oct 11, 2015 16:38 UTC (Sun) by nix (subscriber, #2304) [Link] (3 responses)

> If you can't handle that, and want to be able to send out "no worries" patches, where if you totally screw up, nobody will criticize you, I'm sorry, you are not fit to submit to a project as serious as the Linux kernel.

That is tantamount to saying that most people from huge chunks of the Earth (where the culture militates against public criticism of people and where people will go to great lengths to avoid such criticism) are not fit to submit to "a project as serious as the Linux kernel".

There's nothing wrong with criticizing the *patches*. There's something wrong with criticizing the *person* -- and you claim above that criticizing the person is just fine. That's pretty horrendous (if you meant it, rather than being a sloppy way of phrasing "nobody will criticize your patches". Obviously if the code is crap, you should say so, but that's not the same thing. Everyone totally screws up now and again. This does not reflect on them except to prove that they are human, and attacking them for that is wrong.)

Private vs public

Posted Oct 11, 2015 19:43 UTC (Sun) by lsl (subscriber, #86508) [Link] (2 responses)

> There's nothing wrong with criticizing the *patches*. There's something wrong with criticizing the *person* -- and you claim above that criticizing the person is just fine,

I don't think criticizing a person is generally inappropriate. Sometimes the actual problem is not as much the patches but the mindset that lead to their existence. This is, I think, what Steven's saying.

If you go to great lengths to avoid it, but slip up anyway, well, that happens. The *patch* gets torn apart, you clean it up and everything's fine. But when submitters don't do their due diligence and send crap to the list where it's obvious that they didn't even bother to read the relevant docs, then that's a totally different situation. If you don't even get the most basic stuff right and send one broken patch after another, there's no point in criticizing the individual patches. Until that *person* understands that touching kernel code requires a more careful mindset, looking at the patches is just a waste of time.

So telling those submitters to stop until they change their approach to programming is not only appropriate, but desperately needed. There's more than one way to do that, of course, and no one said you're supposed to scar them for life. With a bit of empathy, you can probably do it without causing all too much pain.

Private vs public

Posted Oct 11, 2015 20:55 UTC (Sun) by neilbrown (subscriber, #359) [Link] (1 responses)

> ...where it's obvious that they didn't even bother to read the relevant docs....

It is *never* obvious what someone else has or hasn't done in private. "They didn't read the relevant docs" may be the simplest explanation that you can think of given your perspective and values.
But people are different and complex. And they change from moment to moment.

Maybe they didn't ready, maybe they read but misunderstood. Maybe they forgot. Maybe they were hasty and careless. Maybe they are having a bad day and they pushed the wrong branch.

That is why attacking the person is such a bad idea. What you are really attacking is your mental model of the person, which is probably quite different in many details to the actual person. You may know what they did publicly,. You never know for certain why. So only attach the actions (i.e the code) not the person.

> If you don't even get the most basic stuff right and send one broken patch after another, there's no point in criticizing the individual patches.

There certainly comes a point where criticizing the patches doesn't seem worth while. But that does not justify attacking the person.

"I'm sorry but I won't be reviewing your patches any more because doing so doesn't appear to help. If you stop posting patches here for at least one month I may then look at future patches you post. If they appear to address my previous review comments, then I may proceed with them"

This talks about what "I" will do, how I see the patches, and specific public actions that you may or may not perform. This is all safe territory. It does not say anything about "you" or the reasons for "your" behaviour. It just identifies the problem and outlines the rules for engagement.

> So telling those submitters to stop until they change their approach to programming

Make that "stop until their patches address the identified issues" and I'll agree with you.

Private vs public

Posted Oct 11, 2015 21:50 UTC (Sun) by neilbrown (subscriber, #359) [Link]

Just a quick addendum...
Modern technological cultures like ours claim to value science over superstition.

Science is about that which is observable and measurable.
Superstition is about whatever we suppose to be the case - untested by measurement.

I'm just suggestion that we should be scientific, not superstitious, in our responses to each other.