2008-07-25 16:28:41 +00:00
|
|
|
#include "cache.h"
|
|
|
|
#include "strbuf.h"
|
|
|
|
#include "run-command.h"
|
2012-11-30 22:41:26 +00:00
|
|
|
#include "sigchain.h"
|
2008-07-25 16:28:41 +00:00
|
|
|
|
2009-10-31 01:44:41 +00:00
|
|
|
#ifndef DEFAULT_EDITOR
|
|
|
|
#define DEFAULT_EDITOR "vi"
|
|
|
|
#endif
|
|
|
|
|
2009-11-12 00:01:27 +00:00
|
|
|
const char *git_editor(void)
|
2008-07-25 16:28:41 +00:00
|
|
|
{
|
Do not use VISUAL editor on dumb terminals
Refuse to use $VISUAL and fall back to $EDITOR if TERM is unset
or set to "dumb". Traditionally, VISUAL is set to a screen
editor and EDITOR to a line-based editor, which should be more
useful in that situation.
vim, for example, is happy to assume a terminal supports ANSI
sequences even if TERM is dumb (e.g., when running from a text
editor like Acme). git already refuses to fall back to vi on a
dumb terminal if GIT_EDITOR, core.editor, VISUAL, and EDITOR are
unset, but without this patch, that check is suppressed by
VISUAL=vi.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-11-11 23:56:07 +00:00
|
|
|
const char *editor = getenv("GIT_EDITOR");
|
|
|
|
const char *terminal = getenv("TERM");
|
|
|
|
int terminal_is_dumb = !terminal || !strcmp(terminal, "dumb");
|
2008-07-25 16:28:41 +00:00
|
|
|
|
|
|
|
if (!editor && editor_program)
|
|
|
|
editor = editor_program;
|
Do not use VISUAL editor on dumb terminals
Refuse to use $VISUAL and fall back to $EDITOR if TERM is unset
or set to "dumb". Traditionally, VISUAL is set to a screen
editor and EDITOR to a line-based editor, which should be more
useful in that situation.
vim, for example, is happy to assume a terminal supports ANSI
sequences even if TERM is dumb (e.g., when running from a text
editor like Acme). git already refuses to fall back to vi on a
dumb terminal if GIT_EDITOR, core.editor, VISUAL, and EDITOR are
unset, but without this patch, that check is suppressed by
VISUAL=vi.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-11-11 23:56:07 +00:00
|
|
|
if (!editor && !terminal_is_dumb)
|
2008-07-25 16:28:41 +00:00
|
|
|
editor = getenv("VISUAL");
|
|
|
|
if (!editor)
|
|
|
|
editor = getenv("EDITOR");
|
|
|
|
|
Do not use VISUAL editor on dumb terminals
Refuse to use $VISUAL and fall back to $EDITOR if TERM is unset
or set to "dumb". Traditionally, VISUAL is set to a screen
editor and EDITOR to a line-based editor, which should be more
useful in that situation.
vim, for example, is happy to assume a terminal supports ANSI
sequences even if TERM is dumb (e.g., when running from a text
editor like Acme). git already refuses to fall back to vi on a
dumb terminal if GIT_EDITOR, core.editor, VISUAL, and EDITOR are
unset, but without this patch, that check is suppressed by
VISUAL=vi.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2009-11-11 23:56:07 +00:00
|
|
|
if (!editor && terminal_is_dumb)
|
2009-11-12 00:01:27 +00:00
|
|
|
return NULL;
|
2008-07-25 16:28:41 +00:00
|
|
|
|
|
|
|
if (!editor)
|
2009-10-31 01:44:41 +00:00
|
|
|
editor = DEFAULT_EDITOR;
|
2008-07-25 16:28:41 +00:00
|
|
|
|
2009-11-12 00:01:27 +00:00
|
|
|
return editor;
|
|
|
|
}
|
|
|
|
|
|
|
|
int launch_editor(const char *path, struct strbuf *buffer, const char *const *env)
|
|
|
|
{
|
|
|
|
const char *editor = git_editor();
|
|
|
|
|
|
|
|
if (!editor)
|
|
|
|
return error("Terminal is dumb, but EDITOR unset");
|
|
|
|
|
2008-07-25 16:28:41 +00:00
|
|
|
if (strcmp(editor, ":")) {
|
2009-12-30 10:56:16 +00:00
|
|
|
const char *args[] = { editor, path, NULL };
|
2012-11-30 22:41:04 +00:00
|
|
|
struct child_process p;
|
launch_editor: propagate signals from editor to git
We block SIGINT and SIGQUIT while the editor runs so that
git is not killed accidentally by a stray "^C" meant for the
editor or its subprocesses. This works because most editors
ignore SIGINT.
However, some editor wrappers, like emacsclient, expect to
die due to ^C. We detect the signal death in the editor and
properly exit, but not before writing a useless error
message to stderr. Instead, let's notice when the editor was
killed by a terminal signal and just raise the signal on
ourselves. This skips the message and looks to our parent
like we received SIGINT ourselves.
The end effect is that if the user's editor ignores SIGINT,
we will, too. And if it does not, then we will behave as if
we did not ignore it. That should make all users happy.
Note that in the off chance that another part of git has
ignored SIGINT while calling launch_editor, we will still
properly detect and propagate the failed return code from
the editor (i.e., the worst case is that we generate the
useless error, not fail to notice the editor's death).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-11-30 22:41:50 +00:00
|
|
|
int ret, sig;
|
2008-07-25 16:28:41 +00:00
|
|
|
|
2012-11-30 22:41:04 +00:00
|
|
|
memset(&p, 0, sizeof(p));
|
|
|
|
p.argv = args;
|
|
|
|
p.env = env;
|
|
|
|
p.use_shell = 1;
|
|
|
|
if (start_command(&p) < 0)
|
|
|
|
return error("unable to start editor '%s'", editor);
|
|
|
|
|
2012-11-30 22:41:26 +00:00
|
|
|
sigchain_push(SIGINT, SIG_IGN);
|
|
|
|
sigchain_push(SIGQUIT, SIG_IGN);
|
|
|
|
ret = finish_command(&p);
|
launch_editor: propagate signals from editor to git
We block SIGINT and SIGQUIT while the editor runs so that
git is not killed accidentally by a stray "^C" meant for the
editor or its subprocesses. This works because most editors
ignore SIGINT.
However, some editor wrappers, like emacsclient, expect to
die due to ^C. We detect the signal death in the editor and
properly exit, but not before writing a useless error
message to stderr. Instead, let's notice when the editor was
killed by a terminal signal and just raise the signal on
ourselves. This skips the message and looks to our parent
like we received SIGINT ourselves.
The end effect is that if the user's editor ignores SIGINT,
we will, too. And if it does not, then we will behave as if
we did not ignore it. That should make all users happy.
Note that in the off chance that another part of git has
ignored SIGINT while calling launch_editor, we will still
properly detect and propagate the failed return code from
the editor (i.e., the worst case is that we generate the
useless error, not fail to notice the editor's death).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-11-30 22:41:50 +00:00
|
|
|
sig = ret + 128;
|
2012-11-30 22:41:26 +00:00
|
|
|
sigchain_pop(SIGINT);
|
|
|
|
sigchain_pop(SIGQUIT);
|
launch_editor: propagate signals from editor to git
We block SIGINT and SIGQUIT while the editor runs so that
git is not killed accidentally by a stray "^C" meant for the
editor or its subprocesses. This works because most editors
ignore SIGINT.
However, some editor wrappers, like emacsclient, expect to
die due to ^C. We detect the signal death in the editor and
properly exit, but not before writing a useless error
message to stderr. Instead, let's notice when the editor was
killed by a terminal signal and just raise the signal on
ourselves. This skips the message and looks to our parent
like we received SIGINT ourselves.
The end effect is that if the user's editor ignores SIGINT,
we will, too. And if it does not, then we will behave as if
we did not ignore it. That should make all users happy.
Note that in the off chance that another part of git has
ignored SIGINT while calling launch_editor, we will still
properly detect and propagate the failed return code from
the editor (i.e., the worst case is that we generate the
useless error, not fail to notice the editor's death).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2012-11-30 22:41:50 +00:00
|
|
|
if (sig == SIGINT || sig == SIGQUIT)
|
|
|
|
raise(sig);
|
2012-11-30 22:41:26 +00:00
|
|
|
if (ret)
|
2008-07-25 16:28:42 +00:00
|
|
|
return error("There was a problem with the editor '%s'.",
|
|
|
|
editor);
|
2008-07-25 16:28:41 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
if (!buffer)
|
2008-07-25 16:28:42 +00:00
|
|
|
return 0;
|
2008-07-25 16:28:41 +00:00
|
|
|
if (strbuf_read_file(buffer, path, 0) < 0)
|
2008-07-25 16:28:42 +00:00
|
|
|
return error("could not read file '%s': %s",
|
|
|
|
path, strerror(errno));
|
|
|
|
return 0;
|
2008-07-25 16:28:41 +00:00
|
|
|
}
|