Re: [kbuild-devel] [PATCH 08/19] scripts: Make cleanfile/cleanpatch warn about long lines

2007-06-05 Thread Sam Ravnborg
On Tue, Jun 05, 2007 at 09:33:35AM +0200, Oleg Verych wrote: Hallo. On Sun, Jun 03, 2007 at 10:47:00PM +0200, Sam Ravnborg wrote: Subject: [PATCH 08/19] scripts: Make cleanfile/cleanpatch warn about long lines From: H. Peter Anvin [EMAIL PROTECTED] Date: Fri, 25 May 2007 17:58:26

Re: [kbuild-devel] [PATCH] Precompiled headers

2007-06-05 Thread Arnd Bergmann
On Tuesday 05 June 2007, Vegard Nossum wrote: You also need to take some care about the gcc arguments that you pass. E.g. passing -DMODULE when building the precompiled header means that you can't use that .pch when you build a file where you don't pass -DMODULE. So does it make sense

Re: [kbuild-devel] [PATCH] Precompiled headers

2007-06-05 Thread Arnd Bergmann
On Tuesday 05 June 2007, Sam Ravnborg wrote: It's stuff like this that worries me.. In the kernel we rely on a lot of things - more than in usual projects. And one thing we have established in the 2.6 kernel is a trust in the build system. If you change something kbuild will recompile

Re: [kbuild-devel] [PATCH 08/19] scripts: Make cleanfile/cleanpatch warn about long lines

2007-06-05 Thread Oleg Verych
On Tue, Jun 05, 2007 at 10:19:59AM +0200, Sam Ravnborg wrote: [] So, there are some new scripts. What if my proposition will be better, so to speak? Any problems i'm willing to fix/enhance. Note: only one copy of the file required. Sym-linked name *diff* or *patch* will process patches.

Re: [kbuild-devel] [PATCH 08/19] scripts: Make cleanfile/cleanpatch warn about long lines

2007-06-05 Thread Oleg Verych
On Tue, Jun 05, 2007 at 04:12:54PM +0200, Sam Ravnborg wrote: On Tue, Jun 05, 2007 at 03:38:34PM +0200, Oleg Verych wrote: On Tue, Jun 05, 2007 at 10:19:59AM +0200, Sam Ravnborg wrote: [] If on the other hand you are proposing a script to clean whitespace damage in the code then git

Re: [kbuild-devel] [PATCH 08/19] scripts: Make cleanfile/cleanpatch warn about long lines

2007-06-05 Thread Oleg Verych
On Tue, Jun 05, 2007 at 04:57:59PM +0200, Oleg Verych wrote: [] expand | while read line do case $line in ++*) echo $line;; $p*) [ ${#line} -gt $w79 ] : ${long:=line} echo $line | sed /^$p/{s_ *\$__;s_^$p$s7${s}_$p${t}_;s_$s7 _${t}_g} ;; *) echo $line;;