[Svnmerge] [PATCH] fix remaining test failures on Windows

Dustin J. Mitchell dustin at zmanda.com
Wed Aug 1 13:31:28 PDT 2007


On Tue, Jul 31, 2007 at 06:30:22PM +0200, Hernan Martinez Foffani wrote:
> > I don't think that collapsing a sequence of \r or \n to a single \n is
> > the right thing to do: it would destroy paragraph formatting.
> 
> I agree.
> 
> >
> > Is a simple "message = message.replace("\r\n", "\n")" enough? If not,
> > why?
> 
> With the .replace tests pass ok in Windows.

But it won't work for Mac newlines.  I just attached an updated version
of my previous patch, correcting a stupid mistake on my part, to the
bug.

If only I had a windows box to test this on :(

http://subversion.tigris.org/nonav/issues/showattachment.cgi/691/fixWindowsLineTermFailures.patch

Dustin

-- 
        Dustin J. Mitchell
        Storage Software Engineer, Zmanda, Inc.
        http://www.zmanda.com/



More information about the Svnmerge mailing list