Live data from Hacker News

“The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

git.kernel.org

101–110 of 116 posts

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#101
post #95
post #72

Earlier quoted context omitted.

Unrelated to the article, but the early British seasons of Kitchen Nightmares were so good. I still enjoy watching the Fox version (and his new show, 24 Hours), but I appreciate those original seasons much more.

You might like The British Baking Show on Netflix. No yelling, everyone is super supportive of each other.

It's definitely near the top of my list! And may be higher after your comment. :)

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#102
post #33

I've submitted many such small corrections to OpenBSD to help keep that standard high. No one will place those contributions on the same level as real development but I like that I've helped. And I know I appreciate it when even the documentation is kept at the same bar. One of the reasons I fell in love with OpenBSD was the commitment to correctness throughout the entire system. This is a lot harder with Linux becau…

Please keep doing that. I'm still hesitant to contribute to tech@ with code and have yet to find a fixable issue in the man pages/FAQ in my own usage. But I do appreciate that the details are oh so right very often.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#103
post #82

Earlier quoted context omitted.

Interesting. I wonder if Maisa has gone on to write any more patches yet!

Or if she got corrupted by the dark side and works for FB now!

FB employees submit loads of interesting kernel patches.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#104

Earlier quoted context omitted.

In my experience, you can read a (large) code base all you want, but until you get in there and start making (breaking) changes, you're not truly going to understand it. I think Linus's quote largely reflects my opinion/experience. A new comer's changes won't always be good or right, but get in there and try things out!

I have been trying to get into that with not much success. It looks like I end up trying to understand the whole thing at once. I don't know how to look at the source code which solves a specific small problem. Is there proper guide by earlier contributors about how to get used to the source and start breaking/fixing things?

I remember my first few commits to big projects were documentation ones. Like —inspect documentation in nodejs, and node —help.

It gives you exposure to code, the dev loop, general idea of how things work. Then one can take on bigger and bigger challenges.

We all gotta start somewhere.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#105
post #35

Earlier quoted context omitted.

I agree on the general concept of your post, however I cannot observe, at least for the things I maintain, that there is this pattern of "people starting small". At least not small in the sense of trivial changes. They maybe start submitting a small PR but that is most of the times already at the level of the contribution that they'll make in the future. So people that will become core contributors will send a small…

Here's my first PR to Rust: https://github.com/rust-lang/rust/pull/4305 Note that it was rejected because I sent it into the wrong branch (we do use master these days) and so I even had to resubmit it https://github.com/rust-lang/rust/pull/4308 I'm about to send in my 972nd PR now. The Rust book is 520 pages; that was the second version, the first of which was ~300 pages. Quite a big difference from +34/-2. I'm at +1…

This is a great real world example. However I would rate your first PR as very good... Documentation improvement, with examples, is something which is very hard to get. Btw in general, I totally agree with the basic idea of trying to merge trivial patches. I go a step forward and I merge "just typos" PRs even if they have the bad effect of breaking other very important PRs sometimes, because a trivial break will be a trivial rebase anyway and typos cannot stay there forever.

From this POV btw Github does a terrible job not telling maintainers: "merging this PR will result in the following PRs being no longer mergeable". Most of the times the list would be empty, making the merging of small PRs a lot more a non brainer.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#106
post #77
post #9

Linus on trivial patches: https://lkml.org/lkml/2004/12/20/255 [ Linux-kernel added back into the cc, because I actually think this is important. ] On Tue, 21 Dec 2004, Jesper Juhl wrote: > Should I just stop attemting to make these trivial cleanups/fixes/whatever patches? are they more noice than gain? am I being a pain to more skilled people on lkml or can you all live with my, sometimes quite ignorant, patches? I…

Linus says: > I think the trivial patches are among the most important ones - exactly because they are the "entry" patches for every new developer. I have been running a fairly successful open source project and this is not exactly my experience. I guess about 70% of pull request I get is simple spell, grammar, white spaces and the likes. Some time folks send this in wholesale because they want their personal favorit…

The only time I've really seen this has been in October, when DO and GH ran their "Hacktoberfest" promotion. Open 5 PRs during October to get a free T-shirt, and some people really just pick random repositories and send random tiny changes. I really liked Hacktoberfest in the past, but this year these people really soured it to me.

E.g. look at 22-52 days old denied PRs on the HTML spec: https://github.com/whatwg/html/pulls?q=is%3Apr+is%3Aclosed

That's an entirely different thing to "serious" small fixes that actually clean up small problems in docs and such.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#107
post #81
post #77

Earlier quoted context omitted.

Linus says: > I think the trivial patches are among the most important ones - exactly because they are the "entry" patches for every new developer. I have been running a fairly successful open source project and this is not exactly my experience. I guess about 70% of pull request I get is simple spell, grammar, white spaces and the likes. Some time folks send this in wholesale because they want their personal favorit…

> takes enormous amount of time to go through each line and each file >My thesis is that there are lots of folks out there just trying to brag to be "contributor" for popular open source projects Do you have an example of this? All I could find in your top 3 repos was one pull request updating the readme and its only 8 lines. https://github.com/sytelus/CryptoJS/pull/3/files

To be fair, he works for Microsoft, its entirely possible he maintains one of their OS repos.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#108

Earlier quoted context omitted.

It demonstrates how Linus is not only a good programmer, but also a good leader. I get the same idea as with the first Gordon Ramsey Kitchen-fixing programmes. Apparently, people said it was insane how he dared to swear on TV. But basically, he notices things are very wrong, makes sure he gets to the bottom of the mess, and hammers the ugly message very publicly into some owner who lost any connection with reality. I…

The US Kitchen Nightmares are completely faked up drama for US audiences. Watch some of the original UK ones and you’ll see how he tries to coach the chefs with comparatively minimal swearing and anger.

I am indeed mostly talking about the UK versions. The US versions had much more drama and much less cooking, but underlying it had the same theme.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#109
post #92
post #31

Earlier quoted context omitted.

This is such a great comment and very true. Looking at the speed and attitude that maintainers handle trivial PRs with can act as an indicator of the project health and will definitely make it more inviting for new contributors!

I am a bit sad that Linus is more known for his occasional frank outbursts than for his consistent encouragement and supportive tone.

Ngl, that's what I know him for the most.

Re: “The 's' is sad”: 4-year-old submits Linux kernel doc patch (2014)

#110

Earlier quoted context omitted.

Someone can be an asshole and a great leader, at the same or different times. Linus himself, to his credit, recently admitted that he's been unreasonably abusive and is trying to learn better.[0] People are complicated. He needn't be either a saint or a demon. [0] https://lkml.org/lkml/2018/9/16/167

The thing is, all Linus rants that I've seen are entirely reasonable. Yes, calling someone stupid is rude, but rude things should be used in extreme circumstances. And when a single developer can negatively affect millions - wait, actually, billions - of people around the world with a stupid patch, that's exactly the case where being rude and aggressive is entirely reasonable.

Linus himself appears to disagree with you, now. You don't need to defend him.
Post reply on HN