Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
MIME-Version: 1.0
In-Reply-To: <00a701d11ed8$8bc8b1d0$a35a1570$@qqmail.nl>
References: <20151113104240.A90313A0AAD@svn01-us-west.apache.org>
 <00a701d11ed8$8bc8b1d0$a35a1570$@qqmail.nl>
From: Julian Foad <julianfoad@apache.org>
Date: Sun, 15 Nov 2015 21:03:16 +0000
Message-ID: 
 <CAEcU=1Y-7ZDyopwAD0RNYGknkBZeAreO7ceEOQUi551RL=oQmA@mail.gmail.com>
Subject: Re: svn commit: r1714181 -
 /subversion/branches/move-tracking-2/subversion/tests/cmdline/svntest/main.py
To: Bert Huijben <bert@qqmail.nl>
Cc: dev <dev@subversion.apache.org>
Content-Type: text/plain; charset=UTF-8

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