git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] pager: set LV=-c alongside LESS=FRSX

From
Anders Kaseorg <andersk@mit.edu>
Date
Jan 30, 2014, 07:15 UTC
Message-ID
<52E9FC23.2030200@mit.edu>
In-Reply-To
<20140107021404.GK3881@google.com>
On 01/06/2014 09:14 PM, Jonathan Nieder wrote:
Show 12 quoted lines
> +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
Previous: Andreas SchwabNext: Jonathan Nieder
Message 5 of 7 in “pager: set LV=-c alongside LESS=FRSX”
  1. pager: set LV=-c alongside LESS=FRSXJonathan Nieder, Jan 7, 2014
  2. Olaf MeeuwissenJan 7, 2014
  3. Junio C HamanoJan 7, 2014
  4. Andreas SchwabJan 7, 2014
  5. Anders KaseorgJan 30, 2014
  6. pager test: make fake pager consume all its inputJonathan Nieder, Jan 30, 2014
  7. Junio C HamanoJan 31, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.