From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.linuxfoundation.org (smtp1.linux-foundation.org [172.17.192.35]) by mail.linuxfoundation.org (Postfix) with ESMTPS id B87DC948 for ; Wed, 14 Sep 2016 19:26:22 +0000 (UTC) Received: from mail2-relais-roc.national.inria.fr (mail2-relais-roc.national.inria.fr [192.134.164.83]) by smtp1.linuxfoundation.org (Postfix) with ESMTPS id CE5031BE for ; Wed, 14 Sep 2016 19:26:21 +0000 (UTC) Date: Wed, 14 Sep 2016 21:26:19 +0200 (CEST) From: Julia Lawall To: Theodore Ts'o In-Reply-To: <20160914143934.4nkdur7ivgywyfrt@thunk.org> Message-ID: References: <20160913194520.GA8071@cloud> <20160913140322.3ccad27c@lwn.net> <4691924.fimvUkKjuv@vostro.rjw.lan> <20160914020332.GA9558@cloud> <1473819862.32273.16.camel@perches.com> <20160914115456.GB22341@kroah.com> <1473863028.32273.28.camel@perches.com> <20160914143205.GA11149@kroah.com> <20160914143934.4nkdur7ivgywyfrt@thunk.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: Joe Perches , ksummit-discuss@lists.linux-foundation.org, ksummit-discuss@lists.linuxfoundation.org Subject: Re: [Ksummit-discuss] [CORE TOPIC] checkpatch/Codingstyle and trivial patch spam List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Wed, 14 Sep 2016, Theodore Ts'o wrote: > On Wed, Sep 14, 2016 at 04:35:06PM +0200, Julia Lawall wrote: > > > > The --force is only required for non-staging code? If it is required for > > staging code, then we will have to document it in the outreachy tutorial, > > and then all the (non-outreachy) newbies who look at the tutorial will > > know about it... > > The whole point of the option is to try to discourage people from > sending white-space (or only style-only) patches. So I'm not so sure > that it would be such a tragedy if people like (for example) Markus > Elfing or Nick Krause don't find out about the --force option right > away. Given that outreachy folks are encouraged to work on staging > code anyway, would it make a difference to them? People other than outreachy applicants may look at the outreachy tutorial. It is even written that way in some cases. I guess that checkpatch could detect whether -f is used on a staging driver and not require --force if that is the case. This would have the other desirable benefit of reducing the chance that outreachy applicants will send patches on non-staging code. julia