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

Reply via email to