From: John Szakmeister Date: Tue, 04 Nov 2014 20:01:12 GMT Subject: [PATCH v2] diff-highlight: exit when a pipe is broken 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 --- 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