Ben Reser wrote on Wed, Dec 12, 2012 at 13:57:30 -0800: > On Tue, Dec 11, 2012 at 5:59 PM, Ben Reser <b...@reser.org> wrote: > > I'd say that replacing '\r' with a '<space>' is wrong. That would > > change the meaning of some properties. E.G. svn:ignore, svn:externals > > which use lines to handle individual records within them. > > To be more explicit, I think you should change CR or CRLF into LF.
I've applied your patch Gabriela, with a tweak to the log message and with the addition of an expected_dumpfile_path parameter. Currently the test checks what Ben said --- which is also consistent with what you and danielsh suggested in earlier emails. Naturally we can change the test's expectations if down the road we decide the correct behaviour is something else. Thanks for the patch! Daniel P.S. This thread was an unusually long one, for a patch that adds about a dozen lines of code.
