login
Header Space

 
 

Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments)

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Randy Dunlap <rdunlap@...>
Cc: Andrew Morton <akpm@...>, Theodore Tso <tytso@...>, Christoph Hellwig <hch@...>, Al Viro <viro@...>, <linux-kernel@...>, <davem@...>
Date: Wednesday, May 21, 2008 - 3:45 pm

[Randy Dunlap - Wed, May 21, 2008 at 12:33:55PM -0700]
| On Wed, 21 May 2008 22:57:25 +0400 Cyrill Gorcunov wrote:
| 
| > [Andrew Morton - Wed, May 21, 2008 at 10:46:44AM -0700]
| > | On Wed, 21 May 2008 08:09:39 -0400 Theodore Tso <tytso@mit.edu> wrote:
| > | 
| > | > On Wed, May 21, 2008 at 06:32:06AM -0400, Christoph Hellwig wrote:
| > | > > On Wed, May 21, 2008 at 01:50:37AM -0700, Andrew Morton wrote:
| > | > > > Oh, what a marvellous way to encourage new contributors that was. Thank
| > | > > > you so much.
| > | > > > 
| > | > > > For the record: Al speaks only for himself and a lack of expressed
| > | > > > disagrement from others should not be taken as agreement.
| > | > > 
| > | > > But I'd like to second the opinion.  This is getting a little too far.
| > | > > We should rather try to at least enforce very basic standards a lot of
| > | > > the crap shoved in doesn't follow instead of wanking around about exact
| > | > > placement of whitespaces.
| > | > 
| > | > The real question is whether people who are wanking about whitespace
| > | > and spelling fixes in comments will graduate to writing real, useful
| > | > patches.  If they won't, there's no point to encouraging them.
| > | > 
| > | 
| > | Guys, get a clue.  It doesn't matter what that person did.  It is the
| > | effect upon *all* other potential developers which is so damaging here.
| > | Not upon this individual.
| > | 
| > 
| > Btw, we have CodingStyle, SubmittingPatches and other, but why don't
| > we have something like KernelNewbieGuide? Don't get me wrong, but
| > there could be written all rules about - what is good to do, what is bad.
| > So a newbiew who wants to be usefull for kernel could read it and decide
| > what should be done. /Don't beat me ;) / And of course I know about
| > kernelnewbie.org but this (even quite short) document could help I think.
| 
| I did a talk on Linux "social engineering" at {OSCON, IEEE NW something,
| and SCALE}.  All about what (not) to do, how to do it, etc.
| Slides for it are available in http://www.xenotime.net/linux/mentor/ .
| 
| 
| ---
| ~Randy
|

Anrew, Randy, you just wrote KernelNewbiesGuide I think ;) There are
lot of internet sites which could help with what-to-do, and linux-mm.org
and kernel.bugzilla.org and more BUT what I've tried to say is that we
really have such document inside kernel tree (it could be really short)
like:

Which patches are not good for sending to LKML
----------------------------------------------

 - Plain whitespace cleanup

Some rules just *known* only if you have an experience in LKML mailing
but if someone is sending _first_ patch he ever made he just doesn't know
that. That is all. So we need not a link to some inet resource BUT written
in file inside of kernel tree. But that is just my (personal) opinion.

		- Cyrill -
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Standard indentation of arguments, Jianjun Kong, (Wed May 21, 1:58 am)
Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard..., Christoph Hellwig, (Wed May 21, 6:32 am)
Re: CFD: linux-wanking@vger.kernel.org, David Miller, (Wed May 21, 3:49 pm)
Re: CFD: linux-wanking@vger.kernel.org, Al Viro, (Wed May 21, 6:03 pm)
Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard..., Cyrill Gorcunov, (Wed May 21, 3:45 pm)
Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard..., Arjan van de Ven, (Thu May 22, 11:29 am)
Re: CFD: linux-wanking@vger.kernel.org, David Miller, (Wed May 21, 4:10 pm)
Re: CFD: linux-wanking@vger.kernel.org, Willy Tarreau, (Wed May 21, 5:25 pm)
Re: CFD: linux-wanking@vger.kernel.org, Stefan Richter, (Wed May 21, 5:44 pm)
Re: CFD: linux-wanking@vger.kernel.org, Willy Tarreau, (Wed May 21, 5:56 pm)
Re: CFD: linux-wanking@vger.kernel.org, David Miller, (Wed May 21, 5:19 am)
Re: [PATCH] Standard indentation of arguments, Jiri Slaby, (Wed May 21, 3:19 am)
Re: [PATCH] Standard indentation of arguments, David Newall, (Wed May 21, 7:35 am)
speck-geostationary