lkml.org 
[lkml]   [2007]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 0/9] Kconfig: cleanup s390 v2.
On Wed, 25 Apr 2007 19:38:23 -0700 Randy Dunlap <randy.dunlap@oracle.com> wrote:

> Dave Jones wrote:
> > On Wed, Apr 25, 2007 at 05:24:47PM -0700, Andrew Morton wrote:
> >
> > > It would be neat if someone could create and maintain a new
> > > scripts/spot-common-mistakes. Feed it a unified diff and it would complain
> > > about newly-added code (and only newly-added code) which has busted
> > > whitespace, adds new semaphores, adds new kernel_thread calls, etc, etc.
> >
> > years and years ago, when the dinosaurs roamed the land, I hacked up..
> > http://janitor.kernelnewbies.org/scripts/ and then left it by the wayside.
> > Some of the checks it did are actually bogus, but I'm happy to pick that
> > up again if there's interest in it being a useful tool.
> >
> > In fact, I should probably munge it together with a similar thing
> > I wrote at http://www.codemonkey.org.uk/projects/findbugs/
> > (Warning: scary regexps)
> >
> > > It would need to be fairly simple and easily-extensible, as I can
> > > imagine quite a few things getting added to it.
> > >
> > > (Imagines a procmail rule which just bounces the email if
> > > spot-common-mistakes failed)
> >
> > or a git checkin rule that refuses to commit if it fails ;-)
>
> Yep, I was going to mention your scripts but you beat me to it.
>
> I'll be glad to help maintain such animals if wanted.
>

wanted ;)

At least, it would be interesting to investigate the usefulness. I suspect
it will prove to be very useful for the little things.

Heck, someone could subscribe a robot to all the mailing lists which sends
nastygrams straight back at people who submit broken patches. We already
need that for tab-replaced and word-wrapped patches. (ok, we have it -
it's called akpm, but being robotic wearies one)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-04-26 05:07    [W:0.146 / U:0.760 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site