linux-kernel - Re: [U-Boot] [RFC PATCH] checkpatch.pl: Add ability to ignore various messages

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1311804639.21169.22.camel@Joe-Laptop>
Date:	Wed, 27 Jul 2011 15:10:39 -0700
From:	Joe Perches <joe@...ches.com>
To:	Wolfgang Denk <wd@...x.de>
Cc:	Andy Whitcroft <apw@...onical.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Graeme Russ <graeme.russ@...il.com>,
	U-Boot Users <u-boot@...ts.denx.de>,
	linux-kernel@...r.kernel.org
Subject: Re: [U-Boot] [RFC PATCH] checkpatch.pl: Add ability to ignore
 various messages

On Wed, 2011-07-27 at 23:17 +0200, Wolfgang Denk wrote:
> Dear Joe,
> In message <b41636a5e5756917b07e50a4b8309dbb092b0a17.1303778411.git.joe@...ches.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@...ches.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@...ches.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@...ches.com>
    Acked-by: Mike Frysinger <vapier@...too.org>
    Cc: Andy Whitcroft <apw@...onical.com>
    Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@...ux-foundation.org>



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/