git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Boyd Lynn Gerber <gerberb@zenez.com>
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	Git List <git@vger.kernel.org>
Subject: Re: git-1.6.2-rc2 problems on t4034-diff-words.sh
Date: Fri, 27 Feb 2009 12:05:44 -0800	[thread overview]
Message-ID: <7vwsbbir1j.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <alpine.LNX.2.00.0902271216090.19082@suse104.zenez.com> (Boyd Lynn Gerber's message of "Fri, 27 Feb 2009 12:24:49 -0700")

Boyd Lynn Gerber <gerberb@zenez.com> writes:

> Does the test suite use bash as it's shell?  I seem to have to use
> bash now to run the test.  I do not remember having to use bash on
> earlier 1.6.0

If your /bin/sh is not POSIX enough, you may be using a different shell
when building the scripted commands in the main Makefile by setting
SHELL_PATH.  The tests are supposed to use the same one as that.

> I have to use bash to run the test or I get
>
> $ ./t4034-diff-words.sh
> ./t4034-diff-words.sh: syntax error at line 52: `(' unexpected

The open parenthesis on line 52 is the one between h and 4 here:

    cat > expect <<\EOF
    <WHITE>diff --git a/pre b/post<RESET>
    <WHITE>index 330b04f..5ed8eff 100644<RESET>
    <WHITE>--- a/pre<RESET>
    <WHITE>+++ b/post<RESET>
    <BROWN>@@ -1,3 +1,7 @@<RESET>
    <RED>h(4)<RESET><GREEN>h(4),hh[44]<RESET>
    <RESET>
    a = b + c<RESET>

    <GREEN>aa = a<RESET>

    <GREEN>aeff = aeff * ( aaa )<RESET>
    EOF

Perhaps your shell has a broken here-text?

  parent reply	other threads:[~2009-02-27 20:07 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-27 18:06 git-1.6.2-rc2 problems on t4034-diff-words.sh Boyd Lynn Gerber
2009-02-27 18:38 ` Johannes Schindelin
2009-02-27 19:24   ` Boyd Lynn Gerber
2009-02-27 19:43     ` Johannes Schindelin
2009-02-27 20:05     ` Junio C Hamano [this message]
2009-02-27 20:44       ` Boyd Lynn Gerber
2009-02-27 20:49         ` Boyd Lynn Gerber

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7vwsbbir1j.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=gerberb@zenez.com \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).