* [PATCH] t/annotate-tests: Use echo & cat instead of sed
@ 2011-05-05 4:43 Brian Gernhardt
2011-05-05 5:39 ` Junio C Hamano
0 siblings, 1 reply; 3+ messages in thread
From: Brian Gernhardt @ 2011-05-05 4:43 UTC (permalink / raw)
To: Git List; +Cc: Junio C Hamano
The use of the sed command "1i No robots allowed" caused the version
of sed in OS X to die with
sed: 1: "1i "No robots allowed"\n": command i expects \ followed by
text
Since this command was just trying to add a single line to the
beginning of the file, do the same with "echo > file.net && cat file
>> file.new"
Unbreaks t8001 and t8002 on OS X 10.6.7
---
t/annotate-tests.sh | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/t/annotate-tests.sh b/t/annotate-tests.sh
index abb1885..c56a77d 100644
--- a/t/annotate-tests.sh
+++ b/t/annotate-tests.sh
@@ -127,7 +127,8 @@ test_expect_success \
test_expect_success \
'an obfuscated email added' \
- 'sed -e "1i No robots allowed" < file > file.new &&
+ 'echo "No robots allowed" > file.new &&
+ cat file >> file.new &&
mv file.new file &&
GIT_AUTHOR_NAME="E" GIT_AUTHOR_EMAIL="E at test dot git" git commit -a -m "norobots"'
--
1.7.5.258.g6ba2f
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] t/annotate-tests: Use echo & cat instead of sed
2011-05-05 4:43 [PATCH] t/annotate-tests: Use echo & cat instead of sed Brian Gernhardt
@ 2011-05-05 5:39 ` Junio C Hamano
2011-05-05 7:43 ` Arjen Laarhoven
0 siblings, 1 reply; 3+ messages in thread
From: Junio C Hamano @ 2011-05-05 5:39 UTC (permalink / raw)
To: Brian Gernhardt; +Cc: Git List, Arjen Laarhoven
An earlier patch from Arjen Laarhoven
From: Arjen Laarhoven <arjen@yaph.org>
Subject: [PATCH] annotate.sh: Fix failing test on OS X
Date: Wed, 4 May 2011 15:36:21 +0200
Message-ID: <1304516181-92364-1-git-send-email-arjen@yaph.org>
seems to attempt to address the same issue but with a twist to deal with
an incomplete line at the end of file.
Arjen, does Brian's patch look good to you, too? It is much less tricky
and should look obviously correct even to people who sees the code in
question for the first time.
Thanks, both.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] t/annotate-tests: Use echo & cat instead of sed
2011-05-05 5:39 ` Junio C Hamano
@ 2011-05-05 7:43 ` Arjen Laarhoven
0 siblings, 0 replies; 3+ messages in thread
From: Arjen Laarhoven @ 2011-05-05 7:43 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Brian Gernhardt, Git List
On Wed, May 04, 2011 at 10:39:55PM -0700, Junio C Hamano wrote:
> An earlier patch from Arjen Laarhoven
>
> From: Arjen Laarhoven <arjen@yaph.org>
> Subject: [PATCH] annotate.sh: Fix failing test on OS X
> Date: Wed, 4 May 2011 15:36:21 +0200
> Message-ID: <1304516181-92364-1-git-send-email-arjen@yaph.org>
>
> seems to attempt to address the same issue but with a twist to deal with
> an incomplete line at the end of file.
>
> Arjen, does Brian's patch look good to you, too? It is much less tricky
> and should look obviously correct even to people who sees the code in
> question for the first time.
Yup, this looks much better than my fix. At least I learned something
about the sed incompatibilities ;-)
Arjen
--
Arjen Laarhoven
The presence of those seeking the truth is infinitely to be preferred to
those who think they've found it.
-- Terry Pratchett, "Monstrous Regiment"
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-05-05 7:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-05 4:43 [PATCH] t/annotate-tests: Use echo & cat instead of sed Brian Gernhardt
2011-05-05 5:39 ` Junio C Hamano
2011-05-05 7:43 ` Arjen Laarhoven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).