[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1446504 - in /subversion/trunk/subversion: include/svn_repos.h libsvn_repos/repos.c svnadmin/svnadmin.c

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Wed, 23 Apr 2014 19:12:08 +0100

Ivan Zhakov <ivan_at_visualsvn.com> writes:

> This code does not handle CR LF which is standard on Windows. So
> 'svnadmin freeze -M' does not work on Windows. Changing code to:
> [[
> paths = svn_cstring_split(opt_state->filedata->data, "\n\r", TRUE, pool);
> ]]
>
> Fixes the problem. Btw it seems there is no tests for 'svnadmin freeze
> -M' case

'-F' rather than '-M'.

I've committed a \r fix. This does mean admins can no longer use \r in
repository names inside the '-F' file but I don't believe that matters.
I've also improved the test.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*
Received on 2014-04-23 20:12:40 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.