From: Anders Kaseorg <andersk@mit.edu>
Date: Thu, 30 Jan 2014 07:15:47 GMT
Subject: Re: [PATCH] pager: set LV=-c alongside LESS=FRSX
Message-ID: <52E9FC23.2030200@mit.edu>
In-Reply-To: <20140107021404.GK3881@google.com>

On 01/06/2014 09:14 PM, Jonathan Nieder wrote:
> +test_expect_success TTY 'LESS and LV envvars are set for pagination' '
> +	(
> +		sane_unset LESS LV &&
> +		PAGER="env >pager-env.out" &&
> +		export PAGER &&
> +
> +		test_terminal git log
> +	) &&
> +	grep ^LESS= pager-env.out &&
> +	grep ^LV= pager-env.out
> +'
> +

On the Ubuntu PPA builders, I’m seeing this new test fail with SIGPIPE 
about half the time:

died of signal 13 at /build/buildd/git-1.9~rc1/t/test-terminal.perl line 33.
not ok 6 - LESS and LV envvars are set for pagination

Although the test seems to work locally for me, I can reproduce a 
similar failure just by running

$ GIT_PAGER='env >pager-env.out' ./test-terminal.perl git log
died of signal 13 at ./test-terminal.perl line 33.

Anders