Skip to content

Commit 944adac

Browse files
committed
Merge branch 'bw/solaris-sed-tr-test-portability'
* bw/solaris-sed-tr-test-portability: t4015: simplify sed command that is not even seen by sed Avoid difference in tr semantics between System V and BSD Change sed i\ usage to something Solaris' sed can handle
2 parents ea06592 + 05e9d90 commit 944adac

File tree

2 files changed

+21
-14
lines changed

2 files changed

+21
-14
lines changed

t/t0008-ignores.sh

Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,14 @@ test_stderr () {
3737
test_cmp "$HOME/expected-stderr" "$HOME/stderr"
3838
}
3939

40+
broken_c_unquote () {
41+
"$PERL_PATH" -pe 's/^"//; s/\\//; s/"$//; tr/\n/\0/' "$@"
42+
}
43+
44+
broken_c_unquote_verbose () {
45+
"$PERL_PATH" -pe 's/ "/ /; s/\\//; s/"$//; tr/:\t\n/\0/' "$@"
46+
}
47+
4048
stderr_contains () {
4149
regexp="$1"
4250
if grep "$regexp" "$HOME/stderr"
@@ -606,12 +614,11 @@ cat <<-EOF >expected-verbose
606614
$global_excludes:2:!globaltwo b/globaltwo
607615
EOF
608616

609-
sed -e 's/^"//' -e 's/\\//' -e 's/"$//' stdin | \
610-
tr "\n" "\0" >stdin0
611-
sed -e 's/^"//' -e 's/\\//' -e 's/"$//' expected-default | \
612-
tr "\n" "\0" >expected-default0
613-
sed -e 's/ "/ /' -e 's/\\//' -e 's/"$//' expected-verbose | \
614-
tr ":\t\n" "\0" >expected-verbose0
617+
broken_c_unquote stdin >stdin0
618+
619+
broken_c_unquote expected-default >expected-default0
620+
621+
broken_c_unquote_verbose expected-verbose >expected-verbose0
615622

616623
test_expect_success '--stdin' '
617624
expect_from_stdin <expected-default &&
@@ -692,12 +699,11 @@ EOF
692699
grep -v '^:: ' expected-all >expected-verbose
693700
sed -e 's/.* //' expected-verbose >expected-default
694701

695-
sed -e 's/^"//' -e 's/\\//' -e 's/"$//' stdin | \
696-
tr "\n" "\0" >stdin0
697-
sed -e 's/^"//' -e 's/\\//' -e 's/"$//' expected-default | \
698-
tr "\n" "\0" >expected-default0
699-
sed -e 's/ "/ /' -e 's/\\//' -e 's/"$//' expected-verbose | \
700-
tr ":\t\n" "\0" >expected-verbose0
702+
broken_c_unquote stdin >stdin0
703+
704+
broken_c_unquote expected-default >expected-default0
705+
706+
broken_c_unquote_verbose expected-verbose >expected-verbose0
701707

702708
test_expect_success '--stdin from subdirectory' '
703709
expect_from_stdin <expected-default &&

t/t4015-diff-whitespace.sh

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ test_expect_success 'another test, with --ignore-space-at-eol' 'test_cmp expect
145145
test_expect_success 'ignore-blank-lines: only new lines' '
146146
test_seq 5 >x &&
147147
git update-index x &&
148-
test_seq 5 | sed "/3/i \\
148+
test_seq 5 | sed "/3/i\\
149149
" >x &&
150150
git diff --ignore-blank-lines >out &&
151151
>expect &&
@@ -155,7 +155,8 @@ test_expect_success 'ignore-blank-lines: only new lines' '
155155
test_expect_success 'ignore-blank-lines: only new lines with space' '
156156
test_seq 5 >x &&
157157
git update-index x &&
158-
test_seq 5 | sed "/3/i \ " >x &&
158+
test_seq 5 | sed "/3/i\\
159+
" >x &&
159160
git diff -w --ignore-blank-lines >out &&
160161
>expect &&
161162
test_cmp out expect

0 commit comments

Comments
 (0)