[PATCH] Change sed i\ usage to something Solaris' sed can handle

2013-11-03 Thread Ben Walton
Solaris' sed was choking on the i\ commands used in t4015-diff-whitespace as it couldn't parse the program properly. Modify two uses of sed that worked in GNU sed but not Solaris' (/usr/bin or /usr/xpg4/bin) to an equivalent form that is handled properly by both. Signed-off-by: Ben Walton

Re: [PATCH] Change sed i\ usage to something Solaris' sed can handle

2013-10-30 Thread Junio C Hamano
Ben Walton bdwal...@gmail.com writes: On Mon, Oct 28, 2013 at 5:39 PM, Andreas Schwab sch...@linux-m68k.org wrote: Ben Walton bdwal...@gmail.com writes: diff --git a/t/t4015-diff-whitespace.sh b/t/t4015-diff-whitespace.sh index 3fb4b97..0126154 100755 --- a/t/t4015-diff-whitespace.sh +++

Re: [PATCH] Change sed i\ usage to something Solaris' sed can handle

2013-10-28 Thread Andreas Schwab
Ben Walton bdwal...@gmail.com writes: diff --git a/t/t4015-diff-whitespace.sh b/t/t4015-diff-whitespace.sh index 3fb4b97..0126154 100755 --- a/t/t4015-diff-whitespace.sh +++ b/t/t4015-diff-whitespace.sh @@ -145,7 +145,8 @@ test_expect_success 'another test, with --ignore-space-at-eol'

Re: [PATCH] Change sed i\ usage to something Solaris' sed can handle

2013-10-28 Thread Ben Walton
On Mon, Oct 28, 2013 at 5:39 PM, Andreas Schwab sch...@linux-m68k.org wrote: Ben Walton bdwal...@gmail.com writes: diff --git a/t/t4015-diff-whitespace.sh b/t/t4015-diff-whitespace.sh index 3fb4b97..0126154 100755 --- a/t/t4015-diff-whitespace.sh +++ b/t/t4015-diff-whitespace.sh @@ -145,7

Re: [PATCH] Change sed i\ usage to something Solaris' sed can handle

2013-10-28 Thread Andreas Schwab
Ben Walton bdwal...@gmail.com writes: It's an escape. Without it, sed throws: The shell removes it before sed can see it. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 And now for something completely different. --

[PATCH] Change sed i\ usage to something Solaris' sed can handle

2013-10-27 Thread Ben Walton
Solaris' sed was choking on the i\ commands used in t4015-diff-whitespace as it couldn't parse the program properly. Modify two uses of sed that worked in GNU sed but not Solaris' (/usr/bin or /usr/xpg4/bin) to an equivalent form that is handled properly by both. Signed-off-by: Ben Walton