Re: [PATCH] stash: Don't paginate by default with list command

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 30, 2011, 19:26 UTC
Message-ID: <7vk49u1z1n.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20110830175748.GA19715@rdrz.de>

Raphael Zimmerer <killekulla@rdrz.de> writes:

Show 9 quoted lines

> On Tue, Aug 30, 2011 at 07:21:18PM +0200, Ingo Brückl wrote:
>> The output of "stash list" is such that piping into a pager
>> normally isn't necessary but annoying, so disable it by default.
>
> Why is the current behaviour annoying?
>
> Here on my (default) setup the pager automatically exits if the entire
> output can be displayed on the screen, as with any other git
> command.

Ben Walton pointed out exactly the same thing, but there needs one bit of caution/consideration. If you tend to use longer and more descriptive branch name (which is encouraged) and create a lazy stash without giving any message, the default description of the stash would need a fairly wide terminal window to fit on one line, e.g.

    "stash@{N}: WIP on some-long-ish-branch-name:" followed by
    "09997df a one line description of the commit your changes are made on"

concatenated on the same line. "S" in "FRSX" (the default value of "LESS" environment variable we give to those who do not have any) is a good thing to have while reviewing long-ish patches via "git log -p", but having a long line that "S" causes to be chopped counts as not being able to show the entire file on the first screen for the purpose of "F", and introduces a minor irritation of having to exit explicitly with a 'q'.