Subject: Re: shell compatibility issues with SunOS 5.10

From: Nguyen Thai Ngoc Duy <[email-masked]>
Date: Wed, 6 May 2009 16:43:19 +1000

On Wed, May 6, 2009 at 4:16 PM, Junio C Hamano <[email-masked]> wrote:
>> 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.

>> 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