Re: [PATCH 1/2] git-svn.perl: consider all ranges for a given merge, instead of only tip-by-tip

Steven Walter <stevenrwalter@gmail.com> wrote:

I'll wait for input on others who understand mergeinfo better than I do to chime in on the actual changes. Some nits on the tests:

> +# Copyright (c) 2010 Steven Walter

2010?

> +test_expect_success 'verify merge commit' 'x=$(git rev-parse HEAD^2) &&
> +	y=$(git rev-parse branch2) &&
> +	[ $x == $y ]

== is a bash-ism, and we prefer 'test'. A single '=' works fine here, but it's probably better to prefix and quote if you're paranoid:

	test "x$x" = "x$y"