Hey Josh,

Hope you're doing well.

I wanted to let you know that this patch of yours, which landed in git 2.10.1, introduced some test failures, seen on macOS.

Let me know if you need any additional information to track these down.

Thanks,
Jeremy


not ok 65 - format-patch default signature
#	
#		git format-patch --stdout -1 | tail -n 3 >output &&
#		signature >expect &&
#		test_cmp expect output
#	

not ok 132 - format-patch --base
#	
#		git checkout side &&
#		git format-patch --stdout --base=HEAD~3 -1 | tail -n 7 >actual &&
#		echo >expected &&
#		echo "base-commit: $(git rev-parse HEAD~3)" >>expected &&
#		echo "prerequisite-patch-id: $(git show --patch HEAD~2 | git patch-id --stable | awk "{print \$1}")" >>expected &&
#		echo "prerequisite-patch-id: $(git show --patch HEAD~1 | git patch-id --stable | awk "{print \$1}")" >>expected &&
#		signature >> expected &&
#		test_cmp expected actual
#	


commit 480871e09ed2e5275b4ba16b278681e5a8c122ae
Author: Josh Triplett <josh@joshtriplett.org>
Date:   Wed Sep 7 18:12:01 2016 -0700

    format-patch: show base info before email signature
    
    Any text below the "-- " for the email signature gets treated as part of
    the signature, and many mail clients will trim it from the quoted text
    for a reply.  Move it above the signature, so people can reply to it
    more easily.
    
    Similarly, when producing the patch as a MIME attachment, the
    original code placed the base info after the attached part, which
    would be discarded.  Move the base info to the end of the part,
    still inside the part boundary.
    
    Add tests for the exact format of the email signature, and add tests
    to ensure that the base info appears before the email signature when
    producing a plain-text output, and that it appears before the part
    boundary when producing a MIME attachment.
    
    Signed-off-by: Josh Triplett <josh@joshtriplett.org>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>