public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox•com>
To: Anders Melchiorsen <mail@cup•kalibalik.dk>
Cc: git@vger•kernel.org, gitster@pobox•com
Subject: Re: [PATCH RESEND] Do not override LESS
Date: Fri, 22 Aug 2008 22:35:40 -0700	[thread overview]
Message-ID: <7vvdxs2t03.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <1219407912-32085-1-git-send-email-mail@cup.kalibalik.dk> (Anders Melchiorsen's message of "Fri, 22 Aug 2008 14:25:12 +0200")

Anders Melchiorsen <mail@cup•kalibalik.dk> writes:

> Passing options to "less" with the LESS environment variable can
> interfere with existing environment variables. There are at least two
> problems, as the following examples show:
>
> 1. Alice is using git with colors. Now she decides to set LESS=i for
> some reason. Suddenly, she sees codes in place of colors because LESS
> is no longer set automatically.
>
> 2. Bob sets GIT_PAGER="less -RS", but does not set LESS. Git sets
> LESS=FRSX before calling $GIT_PAGER. Now Bob wonders why his pager is
> not always paging, when he explicitly tried to clear the F option.

3. Christ has been happily using git with his PAGER set to "less".  He
   suddenly notices that output from git linewraps and the pager does not
   exit when showing a short output, and gets very unhappy.

  reply	other threads:[~2008-08-23  5:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-22 12:25 [PATCH RESEND] Do not override LESS Anders Melchiorsen
2008-08-23  5:35 ` Junio C Hamano [this message]
2008-08-23  8:28   ` Anders Melchiorsen
2008-08-23  9:08     ` Junio C Hamano
2008-08-23 10:21       ` Anders Melchiorsen
2008-08-24  5:28     ` Jonathan Nieder
2008-08-24  5:38       ` [PATCH] Documentation: clarify pager.<cmd> configuration Jonathan Nieder
2008-08-24 18:59       ` [PATCH RESEND] Do not override LESS Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7vvdxs2t03.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox$(echo .)com \
    --cc=git@vger$(echo .)kernel.org \
    --cc=mail@cup$(echo .)kalibalik.dk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox