Re: svn commit: r25692 - trunk/subversion/svn

On 7/9/07, Karl Fogel <kfogel@red-bean.com> wrote:

> Hmmm, don't we have an APR-y way of invoking external commands? I
> realize the system() call was in the code before, I just wonder if
> this isn't an opportunity to upgrade to something more portable. (I
> guess the main advantage of an APR-y way would be that the error code
> would be meaningful.)

Yeah, that thought crossed my mind as well... hmmm... I think we do
something like when we run an external diff3 tool. Ah, yes, it looks
like we have a whole svn_io_run_cmd() infrastructure, which ultimately
calls apr_proc*() functions, manages process i/o, etc.

Maybe 'svn propedit' and the conflict-callback should be using this
code to launch $EDITOR? (Incidentally, I can't figure out where 'svn
commit' is launching the editor!)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 9 19:59:43 2007