Michal Marek <mmarek@xxxxxxx> writes: > git request-pull called inside a terminal prints part of the output to > the terminal and other parts are piped through the pager. Fix this. > > Signed-off-by: Michal Marek <mmarek@xxxxxxx> > --- Having said all that, I noticed this gem in the context in your patch. > git-request-pull.sh | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/git-request-pull.sh b/git-request-pull.sh > index a2cf5b8..b3aaded 100755 > --- a/git-request-pull.sh > +++ b/git-request-pull.sh > @@ -45,13 +45,13 @@ fi > PAGER= > export PAGER > echo "The following changes since commit $baserev:" Looking at ff06c74 (Improve request-pull to handle non-rebased branches, 2007-05-01), it introduced these specifically to address the issue you are bringing up, but it is ineffective. I'd suggest to replace your patch with the attached instead. --- git-request-pull.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/git-request-pull.sh b/git-request-pull.sh index a2cf5b8..f86d8fd 100755 --- a/git-request-pull.sh +++ b/git-request-pull.sh @@ -42,8 +42,8 @@ if [ -z "$branch" ]; then status=1 fi -PAGER= -export PAGER +GIT_PAGER=cat +export GIT_PAGER echo "The following changes since commit $baserev:" git shortlog --max-count=1 $baserev | sed -e 's/^\(.\)/ \1/' -- 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