Re: shell compatibility issues with SunOS 5.10
- From
Nguyen Thai Ngoc Duy <pclouds@gmail.com>
- Date
- May 6, 2009, 06:43 UTC
- Message-ID
- <fcaeb9bf0905052343s7388d057j3d2b219aecc2faeb@mail.gmail.com>
- In-Reply-To
- <7vljpaycwg.fsf@alter.siamese.dyndns.org>
On Wed, May 6, 2009 at 4:16 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 12 quoted lines
>> 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.
Show 15 quoted lines
>> 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