Re: [PATCH v2] diff-highlight: exit when a pipe is broken

2014-11-04 Thread Junio C Hamano
Jeff King writes: > On Tue, Nov 04, 2014 at 03:01:12PM -0500, John Szakmeister wrote: > >> 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 r

Re: [PATCH v2] diff-highlight: exit when a pipe is broken

2014-11-04 Thread Jeff King
On Tue, Nov 04, 2014 at 03:01:12PM -0500, John Szakmeister wrote: > 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

[PATCH v2] diff-highlight: exit when a pipe is broken

2014-11-04 Thread John Szakmeister
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