Skip to content

Commit

Permalink
Avoid using non-portable echo -n in tests.
Browse files Browse the repository at this point in the history
Expecting echo to recognise -n is a BSDism.  Using printf is far more
portable.

Discovered on OS X 10.5.5 in t4030-diff-textconv.sh and changed in all
the test scripts.

Signed-off-by: Brian Gernhardt <[email protected]>
Signed-off-by: Junio C Hamano <[email protected]>
  • Loading branch information
Benabik authored and gitster committed Nov 1, 2008
1 parent df5e91f commit 6ecfd91
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion t/t2005-checkout-index-symlinks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ file if core.symlinks is false.'
test_expect_success \
'preparation' '
git config core.symlinks false &&
l=$(echo -n file | git hash-object -t blob -w --stdin) &&
l=$(printf file | git hash-object -t blob -w --stdin) &&
echo "120000 $l symlink" | git update-index --index-info'

test_expect_success \
Expand Down
4 changes: 2 additions & 2 deletions t/t2102-update-index-symlinks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,12 @@ even if a plain file is in the working tree if core.symlinks is false.'
test_expect_success \
'preparation' '
git config core.symlinks false &&
l=$(echo -n file | git hash-object -t blob -w --stdin) &&
l=$(printf file | git hash-object -t blob -w --stdin) &&
echo "120000 $l symlink" | git update-index --index-info'

test_expect_success \
'modify the symbolic link' '
echo -n new-file > symlink &&
printf new-file > symlink &&
git update-index symlink'

test_expect_success \
Expand Down
2 changes: 1 addition & 1 deletion t/t4030-diff-textconv.sh
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ index ad8b3d2..67be421
EOF
# make a symlink the hard way that works on symlink-challenged file systems
test_expect_failure 'textconv does not act on symlinks' '
echo -n frotz > file &&
printf frotz > file &&
git add file &&
git ls-files -s | sed -e s/100644/120000/ |
git update-index --index-info &&
Expand Down
4 changes: 2 additions & 2 deletions t/t6025-merge-symlinks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ git add file &&
git commit -m initial &&
git branch b-symlink &&
git branch b-file &&
l=$(echo -n file | git hash-object -t blob -w --stdin) &&
l=$(printf file | git hash-object -t blob -w --stdin) &&
echo "120000 $l symlink" | git update-index --index-info &&
git commit -m master &&
git checkout b-symlink &&
l=$(echo -n file-different | git hash-object -t blob -w --stdin) &&
l=$(printf file-different | git hash-object -t blob -w --stdin) &&
echo "120000 $l symlink" | git update-index --index-info &&
git commit -m b-symlink &&
git checkout b-file &&
Expand Down
2 changes: 1 addition & 1 deletion t/t9400-git-cvsserver-server.sh
Original file line number Diff line number Diff line change
Expand Up @@ -424,7 +424,7 @@ cd "$WORKDIR"
test_expect_success 'cvs update (-p)' '
touch really-empty &&
echo Line 1 > no-lf &&
echo -n Line 2 >> no-lf &&
printf "Line 2" >> no-lf &&
git add really-empty no-lf &&
git commit -q -m "Update -p test" &&
git push gitcvs.git >/dev/null &&
Expand Down

0 comments on commit 6ecfd91

Please sign in to comment.