Skip to content

Commit

Permalink
Fix t5510-fetch's use of sed
Browse files Browse the repository at this point in the history
POSIX says sed may add a trailing LF if there isn't already
one there.  We shouldn't rely on it not adding that LF, as
some systems (Mac OS X for example) will add it.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
  • Loading branch information
Shawn O. Pearce authored and Junio C Hamano committed Mar 13, 2007
1 parent 9dc09c7 commit 6016e35
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion t/t5510-fetch.sh
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,13 @@ test_expect_success 'bundle does not prerequisite objects' '
git add file2 &&
git commit -m add.file2 file2 &&
git bundle create bundle3 -1 HEAD &&
sed "1,4d" < bundle3 > bundle.pack &&
(
while read x && test -n "$x"
do
:;
done
cat
) <bundle3 >bundle.pack &&
git index-pack bundle.pack &&
test 4 = $(git verify-pack -v bundle.pack | wc -l)
'
Expand Down

0 comments on commit 6016e35

Please sign in to comment.