From: Junio C Hamano Date: Thu, 21 Feb 2013 18:51:24 GMT Subject: Re: [PATCH v4.1 09/12] sequencer.c: teach append_signoff to avoid adding a duplicate newline Message-ID: <7vip5lv6tv.fsf@alter.siamese.dyndns.org> In-Reply-To: <1360665222-3166-1-git-send-email-drafnel@gmail.com> Brandon Casey writes: > Teach append_signoff to detect whether a blank line exists at the position > that the signed-off-by line will be added, and refrain from adding an > additional one if one already exists. Or, add an additional line if one > is needed to make sure the new footer is separated from the message body > by a blank line. > > Signed-off-by: Brandon Casey > --- > > > A slight tweak. And I promise, no more are coming. When I do $ git commit -s it should start my editor with this in the buffer: ---------------------------------------------------------------- Signed-off-by: Junio C Hamano ---------------------------------------------------------------- and the cursor blinking at the beginning of the file. Annoyingly this step breaks it by removing the leading blank line.