Re: shell compatibility issues with SunOS 5.10

On Wed, May 6, 2009 at 4:16 PM, Junio C Hamano [off-list ref] wrote:

quoted

diff --git a/t/t4200-rerere.sh b/t/t4200-rerere.sh
index b68ab11..61ccdee 100755
--- a/t/t4200-rerere.sh
+++ b/t/t4200-rerere.sh

@@ -57,7 +57,7 @@ test_expect_success 'conflicting merge' '

      test_must_fail git merge first
 '

-sha1=$(sed -e 's/    .*//' .git/MERGE_RR)
+sha1=$(cut -f 1 .git/MERGE_RR)

I do not know why you need this one.  It shouldn't hurt, though.

Well because it wouldn't work without it. But on the other hand, I
grepped "sed.*\t" through and found many of them. Still wondering
while only this fails. I will look at it again when I have time.

quoted

diff --git a/t/t5300-pack-object.sh b/t/t5300-pack-object.sh
index e2aa254..9a916d3 100755
--- a/t/t5300-pack-object.sh
+++ b/t/t5300-pack-object.sh

@@ -315,7 +315,7 @@ test_expect_success 'unpacking with --strict' '

      head -n 10 LIST | git update-index --index-info &&
      LI=$(git write-tree) &&
      rm -f .git/index &&
-     tail -n 10 LIST | git update-index --index-info &&
+     tail -10 LIST | git update-index --index-info &&

I do not know why your "head" apparently accepts -n (see the context) but
not your "tail"; as POSIX frowns upon head/tail -$number, this one is a
regression.

Hey, it's Sun's magic. head manpage mentions -n and it works (I
checked) while tail does not accept it. GNU tail's manpage does not
mention that it supports [+-]N syntax but it seems to work.
-- 
Duy