From: Cyrill Gorcunov <gorcunov@gmail.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Theodore Tso <tytso@mit.edu>,
Christoph Hellwig <hch@infradead.org>,
Al Viro <viro@ZenIV.linux.org.uk>,
linux-kernel@vger.kernel.org, davem@davemloft.net
Subject: Re: CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments)
Date: Wed, 21 May 2008 22:57:25 +0400 [thread overview]
Message-ID: <20080521185725.GD6932@cvg> (raw)
In-Reply-To: <20080521104644.5b6f4e63.akpm@linux-foundation.org>
[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.
- Cyrill -
next prev parent reply other threads:[~2008-05-21 18:57 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-21 5:58 [PATCH] Standard indentation of arguments Jianjun Kong
2008-05-21 7:19 ` Jiri Slaby
2008-05-21 11:35 ` David Newall
2008-05-21 8:34 ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) Al Viro
2008-05-21 8:50 ` Andrew Morton
2008-05-21 9:19 ` CFD: linux-wanking@vger.kernel.org David Miller
2008-05-21 9:41 ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) Al Viro
2008-05-21 17:44 ` Andrew Morton
2008-05-21 19:45 ` Theodore Tso
2008-05-21 20:38 ` Rene Herman
2008-05-21 20:50 ` Andrew Morton
2008-05-21 21:08 ` Rene Herman
2008-05-21 21:12 ` Andrew Morton
2008-05-21 21:32 ` Rene Herman
2008-05-22 19:40 ` Jaya Kumar
2008-05-21 10:32 ` Christoph Hellwig
2008-05-21 12:09 ` Theodore Tso
2008-05-21 17:46 ` Andrew Morton
2008-05-21 18:57 ` Cyrill Gorcunov [this message]
2008-05-21 19:31 ` Andrew Morton
2008-05-21 20:10 ` CFD: linux-wanking@vger.kernel.org David Miller
2008-05-21 21:25 ` Willy Tarreau
2008-05-21 21:44 ` Stefan Richter
2008-05-21 21:56 ` Willy Tarreau
2008-05-21 21:08 ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments) Valdis.Kletnieks
2008-05-21 23:01 ` Adrian Bunk
2008-05-21 23:17 ` Rene Herman
2008-05-21 23:55 ` Adrian Bunk
2008-05-22 0:10 ` Rene Herman
2008-05-22 14:31 ` Adrian Bunk
2008-05-21 23:35 ` Al Viro
2008-05-22 0:38 ` Adrian Bunk
2008-05-22 1:05 ` Alan Cox
2008-05-22 16:08 ` Stefan Richter
2008-05-21 19:33 ` Randy Dunlap
2008-05-21 19:45 ` Cyrill Gorcunov
2008-05-21 19:54 ` Andrew Morton
2008-05-21 20:06 ` Jesper Juhl
2008-05-21 20:27 ` CFD: linux-wanking@vger.kernel.org (was [PATCH] Standard indentation of arguments), " Jonathan Corbet
2008-05-21 20:37 ` Jesper Juhl
2008-05-21 23:46 ` Jesper Juhl
2008-05-22 0:12 ` Adrian Bunk
2008-05-22 0:32 ` Jesper Juhl
2008-05-22 1:03 ` Alan Cox
2008-05-22 14:56 ` Pavel Machek
2008-05-22 15:29 ` Arjan van de Ven
2008-05-22 0:14 ` Al Viro
2008-05-22 0:34 ` Jesper Juhl
2008-05-22 1:07 ` Johannes Weiner
2008-05-22 4:51 ` Cyrill Gorcunov
2008-05-22 5:34 ` david
2008-05-31 16:23 ` Nick Andrew
2008-05-21 19:42 ` Johannes Weiner
2008-05-21 19:47 ` Cyrill Gorcunov
2008-05-21 19:49 ` CFD: linux-wanking@vger.kernel.org David Miller
2008-05-21 22:03 ` Al Viro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080521185725.GD6932@cvg \
--to=gorcunov@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=davem@davemloft.net \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=viro@ZenIV.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox