Re: [PATCH] Quote filename passed to $EDITOR

Scott Lamb <slamb@slamb.org> writes:

> On Tue, Jul 23, 2002 at 11:37:56AM -0500, cmpilato@collab.net wrote:
> > > Why not use a replacement for system that doesn't run through a
> > > shell in the first place, fork()/execve().
> >
> > We've been here. Let it die.
>
> Ehh? This strikes me as a much better way to do it. Below is a patch
> that should do this portably, though please don't apply yet - I think I
> will factor the messiness into a new function and use it for other
> invocations (diff?)

I appreciate the patch, especially since it looks pretty darn close to
the one I committed long ago, and like a mirror image of the one I
committed not-quite-so long ago (when I reverted the first one). :-)

Seriously, we have been here before, in threads as long as the Starr
Report, and in the end we decided to stick with system().

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 23 20:30:37 2002