Re: Extra newline added to commit messages -- why?

On Friday 16 April 2010, Ori Avtalion wrote:
> [...] I realized that the commits I saw without the extra newline
> were either made by "commit -m", git-svn, or by removing the helper
> "--This line, and those below, will be ignored--" text from the
> commit message file and telling the editor to not add a newline to
> the end of a file. (e.g. Can be done using an autocmd in vim)
>
> Why is that newline added in the first place?
>
> Shouldn't Subversion maintain the appearance of the commit message
> as it is in the temp commit message file and ignore the newline
> between the message itself and the "--This line" text?

IMHO, no. The newline is not the start of the "--This line.." but the end of
the previous one, i.e. part of the previous line.

> As another example of why I find this illogical, these two processes
> should produce the same commit log message, but they don't:
>
> This doesn't add a newline at the end
> 1) $ svn commit -m "Fixed something"
>
> This does:
> 2) $ svn commit # then edit:
> Fixed something
> --This line, and those below, will be ignored--
>
> M a
> EOF

I would actually prefer the first variant to automatically add a newline at
the end unless already present, but that goes against the explicit request to
add a message without a newline. And no, these should not produce the same
log messages.

Uli

-- 
ML: http://subversion.tigris.org/mailing-list-guidelines.html
FAQ: http://subversion.tigris.org/faq.html
Docs: http://svnbook.red-bean.com/
Sator Laser GmbH, Fangdieckstraße 75a, 22547 Hamburg, Deutschland
Geschäftsführer: Thorsten Föcking, Amtsgericht Hamburg HR B62 932
**************************************************************************************
Sator Laser GmbH, Fangdieckstraße 75a, 22547 Hamburg, Deutschland
Geschäftsführer: Thorsten Föcking, Amtsgericht Hamburg HR B62 932
**************************************************************************************
           Visit our website at <http://www.satorlaser.de/>
**************************************************************************************
Diese E-Mail einschließlich sämtlicher Anhänge ist nur für den Adressaten bestimmt und kann vertrauliche Informationen enthalten. Bitte benachrichtigen Sie den Absender umgehend, falls Sie nicht der beabsichtigte Empfänger sein sollten. Die E-Mail ist in diesem Fall zu löschen und darf weder gelesen, weitergeleitet, veröffentlicht oder anderweitig benutzt werden.
E-Mails können durch Dritte gelesen werden und Viren sowie nichtautorisierte Änderungen enthalten. Sator Laser GmbH ist für diese Folgen nicht verantwortlich.
**************************************************************************************