Re: [PATCH] pager: default to LESS=FRX not LESS=FRSX

From: Shawn O. Pearce <spearce@spearce.org>
Date: Mar 26, 2007, 08:36 UTC
Message-ID: <20070326083617.GG13247@spearce.org>
In-Reply-To: <7vwt14xvaw.fsf@assigned-by-dhcp.cox.net>

Junio C Hamano <junkio@cox.net> wrote:

Show 9 quoted lines

> Eygene Ryabinkin <rea-git@codelabs.ru> writes:
> 
> > Remove unnecessary '-S' option that enabled less to chop long lines.
> > It used to provide some confusion at least for the 'git diff':
> > "Where are my long lines? Are they missed from the patch?".
> 
> It is not "unnecessary", but only "what _you_ are not used to".
> As this breaks expectentions of people already familiar with
> what git does by default, I cannot take this patch.

Hear hear. I completely agree with Junio. I (and a number of other users that I work with) expect this behevaior by default.

-- 
Shawn.