Hi, On Thu, 26 Jul 2007, Junio C Hamano wrote: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > > > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > > --- > > ... > > @@ -3,90 +3,109 @@ > > test_description='test separate work tree' > > . ./test-lib.sh > > > > +i=1 > > + > > test_rev_parse() { > > - ... > > + name="$1" > > + for option in --is-bare-repository --is-inside-git-dir \ > > + --is-inside-work-tree --show-prefix > > + do > > + shift > > + test_expect_success "$name: $option" \ > > + "test '$1' = \"\$(git rev-parse $option)\"" > > > +i=$(($i+1)) > > +test $i = $STOPI && gdb --args git rev-parse $option > > Eh? Yes, got me there. It was a real PITA to debug that thing, given that the tests were written in the spirit of Malbolge. So the easiest method to debug it was for me to "-i -v" the test, run it again with "STOPI=$n .. -v", and work out what was the intended behaviour, and whether to fix the test or the logic. Will clean up. Ciao, Dscho - 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