Re: [PATCH] fix for interactive merge-callback not supporting spaces in file paths

On Tue, 17 Jul 2007, David Glasser wrote:

> On 7/17/07, Erik Huelsmann <ehuels@gmail.com> wrote:

> > I've looked through it and I'm sad to have to say your approach won't
> > really work: Subversion is a library in a (possibly) multithreaded
> > program. Changing the working dir is not acceptable, because threads
> > my influence each other when doing that, getting in each others way.

This change is to the command-line binary, not to the libraries. We
use apr_filepath_set() in three other places in this source file alone.

...
> It's a tricky little problem. Why don't we tokenize the editor
> argument with apr_tokenize_to_argv and then use something like
> apr_proc_create which takes an argv error instead? That's what
> libsvn_ra_svn does (see its client.c(find_tunnel_agent)).

That sounds like a good approach, assuming it's actually necessary
give that Augie's patch looks fine already.