Skip to content

Commit 60b2a2e

Browse files
committed
Merge branch 'avoid-pipes-in-svn-tests'
It is a good idea in general to avoid pipes in test cases, as it makes things more debuggable to have files to inspect (instead of ephemereal piped data that is long gone). This also seemed to work around a problem where MSYS2' Perl would segfault which may, or may not, still be present today. Signed-off-by: Johannes Schindelin <[email protected]>
2 parents 6508492 + 5e1f62e commit 60b2a2e

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

t/t9001-send-email.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1180,8 +1180,8 @@ test_expect_success $PREREQ 'in-reply-to but no threading' '
11801180
11811181
--in-reply-to="<[email protected]>" \
11821182
--no-thread \
1183-
$patches |
1184-
grep "In-Reply-To: <[email protected]>"
1183+
$patches >out &&
1184+
grep "In-Reply-To: <[email protected]>" out
11851185
'
11861186

11871187
test_expect_success $PREREQ 'no in-reply-to and no threading' '

t/t9116-git-svn-log.sh

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,18 @@ test_expect_success 'setup repository and import' '
4343

4444
test_expect_success 'run log' "
4545
git reset --hard origin/a &&
46-
git svn log -r2 origin/trunk | grep ^r2 &&
47-
git svn log -r4 origin/trunk | grep ^r4 &&
48-
git svn log -r3 | grep ^r3
46+
git svn log -r2 origin/trunk >out &&
47+
grep ^r2 out &&
48+
git svn log -r4 origin/trunk >out &&
49+
grep ^r4 out &&
50+
git svn log -r3 >out &&
51+
grep ^r3 out
4952
"
5053

5154
test_expect_success 'run log against a from trunk' "
5255
git reset --hard origin/trunk &&
53-
git svn log -r3 origin/a | grep ^r3
56+
git svn log -r3 origin/a >out &&
57+
grep ^r3 out
5458
"
5559

5660
printf 'r1 \nr2 \nr4 \n' > expected-range-r1-r2-r4

0 commit comments

Comments
 (0)