Re: git-1.6.2-rc2 problems on t4034-diff-words.sh

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Feb 27, 2009, 19:43 UTC
Message-ID: <alpine.DEB.1.00.0902272041510.6600@intel-tinevez-2-302>
In-Reply-To: <alpine.LNX.2.00.0902271216090.19082@suse104.zenez.com>

Hi,

On Fri, 27 Feb 2009, Boyd Lynn Gerber wrote:

Show 12 quoted lines

> On Fri, 27 Feb 2009, Johannes Schindelin wrote:
> 
> > But as I did not forget your kind words, I'll try to help.
> >
> > > * FAIL 6: use regex supplied by driver
> > >
> > >
> > >                 word_diff --color-words
> 
> 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

No, bash should not be required.

Show 8 quoted lines

> > It might be a regex related issue.  Could you
> >
> > - run the test with -i -v (and if that does not help, "sh -x t...")?
> 
> 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

Could you tell me what your version of t4034 has at line 52? Mine does not have a single parenthesis on that line.

Show 12 quoted lines

> But if I run
> 
> $ bash ./t4034-diff-words.sh
> *   ok 1: setup
> *   ok 2: word diff with runs of whitespace
> *   ok 3: word diff with a regular expression
> *   ok 4: set a diff driver
> *   ok 5: option overrides .gitattributes
> * FAIL 6: use regex supplied by driver
> 
> 
>                 word_diff --color-words

Could you run it again after appending "-i -v" to the command line?

Ciao, Dscho