public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Joe Perches <joe@perches.com>
To: Wolfgang Denk <wd@denx.de>
Cc: Andy Whitcroft <apw@canonical.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Graeme Russ <graeme.russ@gmail.com>,
	U-Boot Users <u-boot@lists.denx.de>,
	linux-kernel@vger.kernel.org
Subject: Re: [U-Boot] [RFC PATCH] checkpatch.pl: Add ability to ignore various messages
Date: Wed, 27 Jul 2011 15:10:39 -0700	[thread overview]
Message-ID: <1311804639.21169.22.camel@Joe-Laptop> (raw)
In-Reply-To: <20110727211707.139ED15794C3@gemini.denx.de>

On Wed, 2011-07-27 at 23:17 +0200, Wolfgang Denk wrote:
> Dear Joe,
> In message <b41636a5e5756917b07e50a4b8309dbb092b0a17.1303778411.git.joe@perches.com> you wrote:
> > Some users would like the ability to not emit some of the
> > messages that checkpatch produces.
> > Add command line option to "--ignore" various message types.
> > Add .checkpatch.conf file to store a default set of "ignore" flags
> > Categorize all ERROR, WARN and CHK types with flags.
> > Signed-off-by: Joe Perches <joe@perches.com>
> > ---
> >  scripts/checkpatch.pl |  456 +++++++++++++++++++++++++++++++++++--------------
> >  1 files changed, 327 insertions(+), 129 deletions(-)
> Did you receive any feedback for your proposal, especially from Andrew
> Morton?
> This feature would be really useful for us, but I'd rather see this in
> mainline than starting a fork.

A better version is in Linus' current 3.0+ as of a couple of days ago.
It's not in 3.0.

commit 000d1cc1829f938c87402fc2fd4bb5e8daed6b52
Author: Joe Perches <joe@perches.com>
Date:   Mon Jul 25 17:13:25 2011 -0700

    checkpatch.pl: add ability to ignore various messages
    
    Some users would like the ability to not emit some of the messages that
    checkpatch produces.  This can make it easier to use checkpatch in other
    projects and integrate into scm hook scripts.
    
    Add command line option to "--ignore" various message types.  Add option
    --show-types to emit the "type" of each message.  Categorize all ERROR,
    WARN and CHK messages with types.
    
    Add optional .checkpatch.conf file to store default options.
    3 paths are searched for .checkpatch.conf
        .             customized per-tree configurations
        $HOME         user global configuration when per-tree configs don't exis
        ./scripts     lk defaults to override script
    The .conf file can contain any valid command-line argument and
    the contents are prepended to any additional command line arguments.
    Multiple lines may be used, blank lines are ignored, # is a comment.
    
    Update "false positive" output for readability.
    
    Update version to 0.32
    
    Signed-off-by: Joe Perches <joe@perches.com>
    Acked-by: Mike Frysinger <vapier@gentoo.org>
    Cc: Andy Whitcroft <apw@canonical.com>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>




  reply	other threads:[~2011-07-27 22:10 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-25  5:36 Expanding checkpatch for non-linux (specifically U-Boot) use Graeme Russ
2011-04-25  6:02 ` Joe Perches
2011-04-25  6:22   ` Graeme Russ
2011-04-25  6:32     ` Joe Perches
2011-04-26  0:43       ` [RFC PATCH] checkpatch.pl: Add ability to ignore various messages Joe Perches
2011-06-09 19:26         ` Joe Perches
2011-06-09 19:40           ` [U-Boot] " Mike Frysinger
2011-07-27 21:17         ` Wolfgang Denk
2011-07-27 22:10           ` Joe Perches [this message]
2011-08-24 21:25             ` Wolfgang Denk
2011-08-24 21:33               ` Joe Perches

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=1311804639.21169.22.camel@Joe-Laptop \
    --to=joe@perches.com \
    --cc=akpm@linux-foundation.org \
    --cc=apw@canonical.com \
    --cc=graeme.russ@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=u-boot@lists.denx.de \
    --cc=wd@denx.de \
    /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