]> git.ipfire.org Git - thirdparty/git.git/commit - column.c
column: show auto columns when pager is active
authorKevin Daudt <me@ikke.info>
Mon, 16 Oct 2017 18:35:11 +0000 (20:35 +0200)
committerJunio C Hamano <gitster@pobox.com>
Tue, 17 Oct 2017 03:16:45 +0000 (12:16 +0900)
commitb2d3fd287b60aae963d683f89f24dc6cb97520f8
tree6251ba6ac1951e7d1a6501b905be693a265dda31
parentff1e72483f416ecb58737205163031f54c24e754
column: show auto columns when pager is active

When columns are set to automatic for git tag and the output is
paginated by git, the output is a single column instead of multiple
columns.

Standard behaviour in git is to honor auto values when the pager is
active, which happens for example with commands like git log showing
colors when being paged.

Since ff1e72483 (tag: change default of `pager.tag` to "on",
2017-08-02), the pager has been enabled by default, exposing this
problem to more people.

finalize_colopts in column.c only checks whether the output is a TTY to
determine if columns should be enabled with columns set to auto. Also
check if the pager is active.

Adding a test for git column is possible but requires some care to work
around a race on stdin. See commit 18d8c2693 (test_terminal: redirect
child process' stdin to a pty, 2015-08-04). Test git tag instead, since
that does not involve stdin, and since that was the original motivation
for this patch.

Helped-by: Rafael Ascensão <rafa.almas@gmail.com>
Signed-off-by: Kevin Daudt <me@ikke.info>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
column.c
t/t7006-pager.sh