On Wed, 2013-07-17 at 08:32 +0800, Jeff Liu wrote: > Another thing might deviated from the main theme, but I'd like to raise it > here because I would like to see what's the proper way for that. > > For instance, people A posted a patch set to the mailing list at first, > people B think that there are some issues in A's implementation, and he > happened to play around the same stuff recently, so he submitted another > patch series. Finally, people B made it. > (In that period, people A kept silent, maybe because he/she was unhappy) > > This is a actual occurrence I once observed from a subsystem list(my > apologies, I just want to talk this case rather than against somebody), > it seems people A is a new comer(because I can not searched any past > commits of him/her from the git log), people B is definitely a senior guy. > > So that's my question, is that a proper collaboration form in kernel > community? Does it better if people B could give some suggestions to > help A to improve the code, especially if those help would help A stepping > into the kernel development -- maybe it's depend largely on one's opinion. :( This is a completely different issue from the one in this thread, but it is also a legitimate issue and honestly, a bigger one than perceived insults. Is it proper collaboration? Absolutely not. Something that I try to be sensitive to as it's something I can do as well. There's been things on my todo list, where someone would send me patches that do it. I would be thinking "darn it, I wanted to do it" and even worse, the patches that were sent wouldn't be of the way I wanted them. But I've tried to be good, and instead of just going about and implementing it myself, I would try to help the person massage the patches into what I wanted. That takes a lot of effort and discipline, and honestly, helping someone else do the work you wanted is much harder than just doing it yourself. Sometimes the maintainer just takes the easier route, and does the work themselves (because it's also more fun too). But that's really a slap in the face of the person that submitted the work in the first place. If anything hurts the community, it's this behavior. Not Linus giving someone an ass wipe. -- Steve -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html