RFR: git-pr: git diff can exit with code 141

Erik Helin ehelin at openjdk.java.net
Fri Feb 7 15:04:22 UTC 2020


Hi all,

please review this small patch that allows `git diff` to exit with code 141
during the `git pr show` command. `git diff` might exit with code 141 if it is
using a pager (e.g. `less`) that works with signals (e.g. `SIGPIPE`). For
example, if a user is viewing a large(r) patch using `git diff` using `less` and
presses "q", then `SIGPIPE` is sent to `git diff` which therefore will exit with
code 128 + 13 = 141 (`SIGPIPE` is signal 13).

The fix is simple, just make `git pr show` allow `git diff` to exit with code
141.

Testing:
- Manual testing of `git pr show`

Thanks,
Erik

-------------

Commits:
 - d9f6fea1: Remove trailing whitespace
 - 6820b2ac: git-pr: git diff can exit with code 141

Changes: https://git.openjdk.java.net/skara/pull/413/files
 Webrev: https://webrevs.openjdk.java.net/skara/413/webrev.00
  Stats: 10 lines in 1 file changed: 7 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/skara/pull/413.diff
  Fetch: git fetch https://git.openjdk.java.net/skara pull/413/head:pull/413

PR: https://git.openjdk.java.net/skara/pull/413


More information about the skara-dev mailing list