Steven Walter <stevenrwalter@xxxxxxxxx> 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" -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html