I think this is great advice. While I've only contributed to a couple open source projects and haven't lead my own large ones, I completely agree. There's more to submitting patches than writing the patch- understanding contribution guidelines (code style, documentation, testing), responding to feedback, etc, these are all extremely important in actually contributing to a project. And having even just that little patch merged in feels great when you're starting out.<p>Plus, no one wants to make a big helpful functional contribution only to be thrown away because they weren't aware of how the community operates. Small patches have a low risk as far as learning about how to contribute, even if the patch is rejected for whatever reason.<p>Though this seems somewhat obvious, it's nice to have it stated and validated by the leadership of one of the largest open source projects.
Great attitude. Linus has had plenty of heated moments on the mailing lists, but 1) he's often right and 2) his venom is usually reserved for experienced developers that really should know better. Glad to him being so open and accepting with newbie kernel hackers.
That makes a lot of sense. If I look at any project I've contributed to, it always started out with something small. Taking the time to make a small PR that makes a tiny improvement:<p>- shows you care about improving the project; you took the time to improve something small that others would ignore<p>- lets you test the waters. Are the project owners receptive of changes? Is it pleasant trying to contribute to the project? Or do they ignore your patch and don't reply for 2 weeks. You wouldn't want to spend a lot of effort just to find that out.<p>- gives you a chance to become familiar with the process, the tools, gain practice, and hopefully get rewarded with your change being accepted<p>For example, my first CL to Go was a trivial change:<p><a href="https://codereview.appspot.com/97280043/patch/40001/50001" rel="nofollow">https://codereview.appspot.com/97280043/patch/40001/50001</a><p>But without having done that (and having a good experience) I couldn't have been working on more complicated changes like <a href="https://codereview.appspot.com/142360043/" rel="nofollow">https://codereview.appspot.com/142360043/</a> now.
Seems like it's good to remind people every now and then that Linus is a human with empathy, since a lot of them really don't understand the reasons for when he rants, and then start to draw conclusions.<p>On a related note, Theo de Raadt rants need far more attention than they currently get.
Small patches are the "hello world" of open source. They give you a chance to get familiar with and work through the contribution pipeline before you push something sizeable through.<p>There's little useful about "hello world" <i>as a program</i>, but it ensures you've got your toolchain working correctly, which is a necessary precondition for doing real work. Trivial patches are like that.
Reminds me of Lawrence of Arabia. "Big things have small beginnings" I've contributed tiny bug fixes and the feeling was great. I did that, me.
Is it just me or is the main reason this is being upvoted so much is because Linus said something without a bad attitude? This isn't a rhetorical question, I honestly want to know.
I completely agree. The only thing that's topped getting a tiny patch of my own accepted (into pylibtiff) was someone getting me to put a python module I'd written* onto pypi. Both were great feelings, and it's extremely important to encourage such highs in new people contributing to open projects (Linux kernel or random bibtex editor alike.)<p>* the library was in fact just a python wrapper for some C++ code, but that's where leverage for the original author starts, in bindings.