As this is not limited to diff command at all, let's do this instead. -- >8 -- Document use of pager means you will see exit code from the pager Whenever we run pager (either a subcommand that implies use of pager by default, or by explicit request with "git -p cmd"), the main git process becomes the upstream of the pipe that feed the pager, and the exit code from the command as a whole comes from the pager. Long time users may have already got used to this without being documented, but it should be documented. We may be swapping the process ordering in the future so that the exit code from the main git process is always exposed, and at that point this comment should be removed. Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> --- Documentation/git.txt | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/Documentation/git.txt b/Documentation/git.txt index b1cb972..d6ca400 100644 --- a/Documentation/git.txt +++ b/Documentation/git.txt @@ -150,7 +150,9 @@ help ...`. -p:: --paginate:: - Pipe all output into 'less' (or if set, $PAGER). + Pipe all output into 'less' (or if set, $PAGER). Note that this + implies that the exit code you see from the command will be that + of the pager, not git. --no-pager:: Do not pipe git output into a pager. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
