Thread (5 messages) flat view 5 messages, 3 authors, 2016-06-15

Re: [PATCH] git-svn: Allow using arguments to the editor.

From: Thomas Rast <hidden>
Date: 2016-06-15 22:46:14

Gerfried Fuchs wrote:
When setting the EDITOR or VISUAL environment variable, one might want
to hand over arguments (like e.g. for not backgrounding a GUI editor but
waiting for it to finish. This patch enables that posibility, before it
did look for a program with the content of the variable, including the
space as filename part. The change is in sync with regular behavior with
various other tools, git itself included.
[...]
+		my (@editor) = split /\s+/, $editor;
This doesn't handle quoted spaces and such.  launch_editor() seems to
pass the $EDITOR through 'sh -c' if there are any special characters
inside, wouldn't that be appropriate for git-svn too?

-- 
Thomas Rast
trast@{inf,student}.ethz.ch

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help