Thank you! -Manish Goregaokar On Sun, Nov 24, 2019 at 8:51 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > "Manish Goregaokar via GitGitGadget" <gitgitgadget@xxxxxxxxx> > writes: > > > + test_cmp ../expect ../actual && > > Note that "test_cmp expect actual" highlights a breakage, when it > finds one, by showing a diff that shows "what we saw is different > from what we expected to see in this way". So this invocation is > good. > > > + git -C ../submod checkout HEAD^ && > > + git submodule status >../output && > > + awk "{print \$1}" <../output >../actual2 && > > + cd .. && > > + git submodule status >output && > > + awk "{print \$1}" <output >expect2 && > > + test_cmp actual2 expect2 && > > But this one is probably the other way around. I'll fix this up > while queuing, so if there is no other changes necessary, there is > no need to resend the patch. > > > + ! test_cmp actual actual2 > > And the order of this one actually does not matter ;-) > > > + ) > > +' > > Thanks.