Re: [PATCH] t5531: further "matching" fixups

2014-01-10 Thread Jeff King
On Fri, Jan 10, 2014 at 03:34:59PM -0800, Junio C Hamano wrote: > Jeff King writes: > > > ... but the > > failing test is actually somewhat broken in 'next' already. > > Hmph, in what way? I haven't seen t5531 breakage on 'next', with or > without your series... The test still passes, but it

Re: [PATCH] t5531: further "matching" fixups

2014-01-10 Thread Junio C Hamano
Jeff King writes: > ... but the > failing test is actually somewhat broken in 'next' already. Hmph, in what way? I haven't seen t5531 breakage on 'next', with or without your series... > fixes it, and should be done regardless of the other series. > > t/t5531-deep-submodule-push.sh | 1 + > 1

[PATCH] t5531: further "matching" fixups

2014-01-08 Thread Jeff King
Commit 43eb920 switched one of the sub-repository in this test to matching to prepare for a world where the default becomes "simple". However, the main repository needs a similar change. We did not notice any test failure when merged with b2ed944 (push: switch default from "matching" to "simple",