Hi Lawrence, Lawrence Siebert <lawrencesiebert@xxxxxxxxx> writes: > +test_expect_success 'log --count' ' > + git log --count HEAD > actual && > + git rev-list --count HEAD > expect && Why the huge space here between 'git' and 'rev-list'? Also no space after the redirection ('>'), it should be '>actual' and '>expect'. > + test_cmp expect actual > +' > + In the mail I see some trailing whitespace after the last two lines. I can't apply it myself before some time, so I can't see if 'git am' warns me about trailing whitespace. You can use 'git log --check' to see the trailing whitespace in your commits. I'll look if I see some trailing whitespace in your other parts of the patch but I might miss some. Rémi -- 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