Re: scripts/config: fix variable substitution command

2013-09-14 Thread Clément Chauplannaz
2013/9/14 Sedat Dilek : > Hi, > > The ChangeLog from [1] says: > > Commit 229455bc02b87f7128f190c4491b4ce38648 accidentally changed > the separator between sed `s' command and its parameters from ':' to > '/'. > Revert this change. > > Reported-and-tested-by: Linus Walleij > Signed-off-by:

Re: scripts/config: fix variable substitution command

2013-09-14 Thread Clément Chauplannaz
2013/9/14 Sedat Dilek sedat.di...@gmail.com: Hi, The ChangeLog from [1] says: Commit 229455bc02b87f7128f190c4491b4ce38648 accidentally changed the separator between sed `s' command and its parameters from ':' to '/'. Revert this change. Reported-and-tested-by: Linus Walleij

Re: [PATCH 2/4] scripts/config: use sed's POSIX interface

2013-09-13 Thread Clément Chauplannaz
On Sep 13, 2013, at 11:32 AM, Linus Walleij wrote: > On Fri, Sep 13, 2013 at 10:38 AM, Clément Chauplannaz > wrote: > >> Thank you for this report. I was able to reproduce this bug and fix it. > > Thanks! Tested and works fine. Glad to read the patch solves your issue.

Re: [PATCH 2/4] scripts/config: use sed's POSIX interface

2013-09-13 Thread Clément Chauplannaz
2013/9/12 Linus Walleij : > On Thu, Aug 15, 2013 at 11:17 PM, Yann E. MORIN > wrote: > >> From: Clement Chauplannaz >> >> Script `config' relies on extensions of `GNU sed', and is thus not >> working on all Unixes: >> - in-place edition of files (-i), which can be replaced with >> a

Re: [PATCH 2/4] scripts/config: use sed's POSIX interface

2013-09-13 Thread Clément Chauplannaz
2013/9/12 Linus Walleij linus.wall...@linaro.org: On Thu, Aug 15, 2013 at 11:17 PM, Yann E. MORIN yann.morin.1...@free.fr wrote: From: Clement Chauplannaz chaup...@gmail.com Script `config' relies on extensions of `GNU sed', and is thus not working on all Unixes: - in-place edition of

Re: [PATCH 2/4] scripts/config: use sed's POSIX interface

2013-09-13 Thread Clément Chauplannaz
On Sep 13, 2013, at 11:32 AM, Linus Walleij linus.wall...@linaro.org wrote: On Fri, Sep 13, 2013 at 10:38 AM, Clément Chauplannaz chaup...@gmail.com wrote: Thank you for this report. I was able to reproduce this bug and fix it. Thanks! Tested and works fine. Glad to read the patch solves