launch_editor: propagate signals from editor to git
authorJeff King <peff@peff.net>
Fri, 30 Nov 2012 22:41:50 +0000 (17:41 -0500)
committerJunio C Hamano <gitster@pobox.com>
Sun, 2 Dec 2012 10:07:08 +0000 (02:07 -0800)
commit1250857c6c2695020bce6669a4ff43d57a507d91
tree35e0444e367f30239f161d78c4d483f8ae568991
parenta2767c5c91ebda3c083419c80a7f64248c5ec175
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>
editor.c