[Mono-list] newline and cvs
Paolo Molaro
lupus@ximian.com
Sun, 3 Mar 2002 10:27:29 +0100
To anyone who has write access to the cvs repository:
DO NOT CHANGE THE NEWLINE CONVENTION OF EXISTING FILES
Unix and DOS use different conventions for line separators:
\n for Unix and \r\n for DOS (ok, \r for the mac, too).
Currently some files in the class library use \n and some use \r\n;
this is not a problem per se, but I agree it would be good to be
consistent about it for _new_ files (I suggest the unix convention).
However, existing files should *not* be changed to match the convention
used by the last programmer that committed to the file! This makes the
cvs history useless and reviewing patches almost impossible.
So, use an editor that doesn't change the newline convention and review
your patches before committing for errors like this (you should check
the diff anyway before committing).
Thanks.
lupus
--
-----------------------------------------------------------------
lupus@debian.org debian/rules
lupus@ximian.com Monkeys do it better