Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Robert P. J. Day
On Sat, 21 Jul 2007, Mike Frysinger wrote: > On 7/21/07, Sam Ravnborg <[EMAIL PROTECTED]> wrote: > > On Sat, Jul 21, 2007 at 06:03:00PM -0400, Mike Frysinger wrote: > > > On 7/21/07, Sam Ravnborg <[EMAIL PROTECTED]> wrote: > > > >I would much more prefer this functionality to be integrated into >

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Mike Frysinger
On 7/21/07, Sam Ravnborg <[EMAIL PROTECTED]> wrote: > On Sat, Jul 21, 2007 at 06:03:00PM -0400, Mike Frysinger wrote: > > On 7/21/07, Sam Ravnborg <[EMAIL PROTECTED]> wrote: > > >I would much more prefer this functionality to be integrated into unifdef. > > >There is no good reason to have two diff

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Sam Ravnborg
On Sat, Jul 21, 2007 at 06:03:00PM -0400, Mike Frysinger wrote: > On 7/21/07, Sam Ravnborg <[EMAIL PROTECTED]> wrote: > >On Sat, Jul 21, 2007 at 03:21:43PM -0400, Mike Frysinger wrote: > >> On 7/21/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >> >On Sat, Jul 21, 2007 at 04:27:31AM -0400, Mike Frysi

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Mike Frysinger
On 7/21/07, Sam Ravnborg <[EMAIL PROTECTED]> wrote: > On Sat, Jul 21, 2007 at 03:21:43PM -0400, Mike Frysinger wrote: > > On 7/21/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > > >On Sat, Jul 21, 2007 at 04:27:31AM -0400, Mike Frysinger wrote: > > >[] > > >> if you want to make some micro optimizatio

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Sam Ravnborg
On Sat, Jul 21, 2007 at 03:21:43PM -0400, Mike Frysinger wrote: > On 7/21/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > >On Sat, Jul 21, 2007 at 04:27:31AM -0400, Mike Frysinger wrote: > >[] > >> if you want to make some micro optimization in the build install step, > >> sure ... but functionally, t

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Mike Frysinger
On 7/21/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > On Sat, Jul 21, 2007 at 04:27:31AM -0400, Mike Frysinger wrote: > [] > > if you want to make some micro optimization in the build install step, > > sure ... but functionally, the difference is irrelevant considering > > sed operates only on indiv

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Oleg Verych
On Sat, Jul 21, 2007 at 04:27:31AM -0400, Mike Frysinger wrote: [] > if you want to make some micro optimization in the build install step, > sure ... but functionally, the difference is irrelevant considering > sed operates only on individual lines That was an attempt to support less sucking use

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Oleg Verych
On Sat, Jul 21, 2007 at 10:23:44AM +0200, Andreas Schwab wrote: > Oleg Verych <[EMAIL PROTECTED]> writes: > > >> + -e > >> "s/[[:space:]]__user[[:space:]]\{1,\} > > > > substitute one or more ' __user ' > > Substitute ' __user' followed by one or more ' '. \{\} applies only to > the la

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Mike Frysinger
On 7/21/07, Oleg Verych <[EMAIL PROTECTED]> wrote: > * Date: Tue, 17 Jul 2007 16:08:54 +0200 > > > > From: Mike Frysinger <[EMAIL PROTECTED]> > > > > The sed expression used at the moment in scripts/Makefile.headersinst > > relies on the (handy) GNU extension where you can escape ERE's in an > > ot

Re: [kbuild-devel] [PATCH 25/33] kbuild: use POSIX BRE in headers install target

2007-07-21 Thread Oleg Verych
* Date: Tue, 17 Jul 2007 16:08:54 +0200 > > From: Mike Frysinger <[EMAIL PROTECTED]> > > The sed expression used at the moment in scripts/Makefile.headersinst > relies on the (handy) GNU extension where you can escape ERE's in an > otherwise BRE without using the GNU -r option. The following patch