git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).