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

Re: svn commit: r1714181 - /subversion/branches/move-tracking-2/subversion/tests/cmdline/svntest/main.py

From: Julian Foad <julianfoad_at_apache.org>
Date: Sun, 15 Nov 2015 21:03:16 +0000

Bert Huijben wrote:
> Julian Foad wrote:
> > URL: http://svn.apache.org/viewvc?rev=1714181&view=rev
[...]
> > def run_svnmover(*varargs):
> > """Run svnmover with VARARGS, returns exit code as int; stdout, stderr as
> > list of lines (including line terminators). Use binary mode for output."""
> > - return run_command(svnmover_binary, 1, True,
> > + return run_command(svnmover_binary, 1, False,
> > *(_with_auth(_with_config_dir(varargs))))
>
> You missed changing the comment here that documents that you explicitly use binary output.

Thanks. Fixed in r1714499.

- Julian
Received on 2015-11-15 22:03:38 CET

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.