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

On 7/9/07, C. Michael Pilato <cmpilato@collab.net> wrote:
> Garrett Rooney wrote:
> > IIRC there was a specific reason we used system(), but I can't recall
> > the details. Perhaps the archives would tell us...
>
> Yeah, I remember the same. Crawling the archives makes me think it had
> something to do with supporting editors with spaces in their command-lines
> (e.g., emacs -nw).

Hmm. This implies that we need to be careful about not passing files
with spaces or other shell metacharacters in their names as either the
"path" to svn_cl__edit_file_externally or as the "prefix" to
svn_cl__edit_string_internally.

A quick glance over the code implies that we're safe for now, though
(since the merge_file being edited with (e) has a boring temporary
filename).

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org