From c0459ca4dc47edf5d63850a34a5f67febd966f3d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=B6rn=20Engel?= Date: Mon, 21 Apr 2014 16:46:22 -0400 Subject: pager: do allow spawning pager recursively MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit 88e8f908f2b0c56f9ccf8134d8ff9f689af9cc84, which tried to allow GIT_PAGER="git -p column --mode='dense color'" git -p branch and still wanted to avoid "git -p column" to invoke itself. However, this falls into "don't do that -p then" category. In particular, inside "git log", with results going through less, a potentially interesting commit may be found and from there inside "less", the user may want to execute "git show ". Before the commit being reverted, this used to show the patch in less but it no longer does. Signed-off-by: Jörn Engel Reviewed-by: Jeff King Reviewed-by: Matthieu Moy Acked-by: Duy Nguyen Signed-off-by: Junio C Hamano diff --git a/pager.c b/pager.c index 0cc75a8..53670a6 100644 --- a/pager.c +++ b/pager.c @@ -64,7 +64,7 @@ void setup_pager(void) { const char *pager = git_pager(isatty(1)); - if (!pager || pager_in_use()) + if (!pager) return; /* -- cgit v0.10.2-6-g49f6