Re: t5801-remote-helpers.sh fails

On Sat, Nov 10, 2012 at 8:20 PM, Torsten Bögershausen [off-list ref] wrote:

On 11/10/2012 08:15 PM, Felipe Contreras wrote:

quoted

Hi,

On Sat, Nov 10, 2012 at 2:48 PM, Torsten Bögershausen [off-list ref]
wrote:

quoted

on peff/pu t5801 fails, the error is in git-remote-testgit, please see
below.

That's on my Mac OS X box.

I haven't digged further into the test case, but it looks as if
"[-+]A  make NAMEs associative arrays"
is not supported on this version of bash.
/Torsten


/Users/tb/projects/git/git.peff/git-remote-testgit: line 64: declare: -A:
invalid option
declare: usage: declare [-afFirtx] [-p] [name[=value] ...]
/Users/tb/projects/git/git.peff/git-remote-testgit: line 66:
refs/heads/master: division by 0 (error token is "/master")
error: fast-export died of signal 13
fatal: Error while running fast-export

What is your bash --version?

 bash --version
GNU bash, version 3.2.48(1)-release (x86_64-apple-darwin10.0)
Copyright (C) 2007 Free Software Foundation, Inc.

I see, that version indeed doesn't have associative arrays.

On the other hand, Documentation/CodingGuidelines says:
  - No shell arrays.

Yeah, for shell code I guess, but this is bash code.

Could we use perl to have arrays?

I think the code in perl would be harder to follow, and this is meant
not only as a test, but also as a reference.

I'm not exactly sure what we should do here:

a) remove the code (would not be so good as a reference)
b) enable the code conditionally based on the version of bash (harder to read)
c) replace the code without associative arrays (will be much more
complicated and ugly)
d) add a check for the bash version to the top of the test in t/

I'm leaning towards d), followed by b).

If only there was a clean way to do this, so c) would not be so ugly.

After investigating different optins this seems to be the best:

	join -e empty -o '0 1.2 2.2' -a 2 <(echo "$before") <(echo "$after")
| while read e a b; do
		test "$a" == "$b" && continue
		echo "changed $e"
	done

But to me seems a bit harder to grasp. Not sure.

Cheers.

-- 
Felipe Contreras