[PATCH v2] diff-highlight: exit when a pipe is broken
- From
John Szakmeister <john@szakmeister.net>
- Date
- Nov 4, 2014, 20:01 UTC
- Message-ID
- <1415131272-89441-1-git-send-email-john@szakmeister.net>
While using diff-highlight with other tools, I have discovered that Python ignores SIGPIPE by default. Unfortunately, this also means that tools attempting to launch a pager under Python--and don't realize this is happening--means that the subprocess inherits this setting. In this case, it means diff-highlight will be launched with SIGPIPE being ignored. Let's work with those broken scripts by restoring the default SIGPIPE handler.
Signed-off-by: John Szakmeister <john@szakmeister.net> --- Incorporates feedback from Jeff King and now we just restore the default signal handler using the correct case of 'DEFAULT'.
contrib/diff-highlight/diff-highlight | 4 ++++ 1 file changed, 4 insertions(+)
diff --git a/contrib/diff-highlight/diff-highlight b/contrib/diff-highlight/diff-highlight index c4404d4..69a652e 100755 --- a/contrib/diff-highlight/diff-highlight +++ b/contrib/diff-highlight/diff-highlight @@ -14,6 +14,10 @@ my @removed; my @added; my $in_hunk; +# Some scripts may not realize that SIGPIPE is being ignored when launching the +# pager--for instance scripts written in Python. +$SIG{PIPE} = 'DEFAULT'; + while (<>) { if (!$in_hunk) { print;
-- 2.0.1