All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felipe Contreras <felipe.contreras@gmail.com>
To: "Randall S. Becker" <rsbecker@nexbridge.com>,
	'Felipe Contreras' <felipe.contreras@gmail.com>,
	git@vger.kernel.org
Cc: "'Junio C Hamano'" <gitster@pobox.com>,
	git@vger.kernel.org,
	"'Ævar Arnfjörð Bjarmason'" <avarab@gmail.com>,
	"'Jeff King'" <peff@peff.net>
Subject: RE: How dow we educate our users to configure less?
Date: Mon, 28 Jun 2021 16:09:02 -0500	[thread overview]
Message-ID: <60da3a6e157de_1de4220874@natae.notmuch> (raw)
In-Reply-To: <029a01d76c60$24ad0ce0$6e0726a0$@nexbridge.com>

Randall S. Becker wrote:
> On June 28, 2021 4:51 PM, Felipe Contreras wrote:
> >To: Randall S. Becker <rsbecker@nexbridge.com>; 'Felipe Contreras' <felipe.contreras@gmail.com>; git@vger.kernel.org
> >Cc: 'Junio C Hamano' <gitster@pobox.com>; git@vger.kernel.org; 'Ævar Arnfjörð Bjarmason' <avarab@gmail.com>; 'Jeff King'
> ><peff@peff.net>
> >Subject: RE: How dow we educate our users to configure less?
> >
> >Randall S. Becker wrote:
> >> On June 28, 2021 4:18 PM, Felipe Contreras wrote:
> >> >Randall S. Becker wrote:
> >
> >> >I'm saying the **opposite**. I'm saying this should be done in builtin/help.c *not* .profile.
> >> >
> >> >> Admittedly, I am in a highly complex situation, but it is a real
> >> >> one (ok, two because of a diverged path between NonStop and MVS)
> >> >> and there are hundreds in a similar situation.
> >> >
> >> >My patch [1] should work in all your environments.
> >>
> >> Your patch will work in the environments but not in the use case I
> >> tried to explain. I do not want a single configuration of less colours
> >> in .git/config or ~/.gitconfig. That is not going to work in my
> >> situation. I have multiple less colour values that would apply within
> >> a given arbitrary timeframe. The configuration depends on the specific
> >> terminal type set in the environment, either dumb, vt220, t653x,
> >> xterm, cygwin, all of which may happy in short succession. I do not
> >> expect it to be practical to change my git settings to conform to this
> >> patch, so I am trying to point out that I do not see how it can solve
> >> my issue.
> >
> >Are you talking about color settings? If so, what are the values of
> >LESS_TERMCAP_* that you have configured?
> >
> >> The current support, using the TERM environment variable, which is
> >> passed to git in all situations either by the system itself on through
> >> scripts as is the case with Jenkins, is mostly sufficient for less and
> >> git to find its appropriate termcap on all platforms that I use on an
> >> ongoing basis (Windows Cygwin, NonStop OSS, NonStop GUARDIAN, Ubuntu,
> >> MacOS, MVS, USS, Jenkins). The NonStop GUARDIAN environment does
> >> present some paging issues that do not work correctly in some cases
> >> with some terminal emulators, but that's the emulator's problem, not
> >> the termcap specifically.
> >>
> >> So what am I missing?
> >
> >You still have not explained why this would not work on any of your
> >environments:
> >
> >  setenv("LESS_TERMCAP_md", GIT_COLOR_BOLD_RED, 0);
> >  setenv("LESS_TERMCAP_me", GIT_COLOR_RESET, 0);
> 
> I am not saying it will not work technically. Suppose I have a
> terminal session using t653x, which is not vt220 compatible - meaning
> it does not use GIT_COLOR_BOLD_RED or even have the concept of bold
> red) and do a git log or git help. I have another session using vt220,
> which works as configured. I have a third session running in Jenkins
> that keeps things up to date and is a terminal type dumb.

Does `git grep` show colors?

> Your patch appears to imply that I need to run git config to change
> the values associated with colours to make things work, correct?

No, you just need to enable color.man

  git -c color.man=true help git

> So how do the three sessions all work simultaneously or do I only get
> to use one of them at a time and reconfigure when I want to use a
> different terminal?

Do you need to reconfigure anything for `git grep` to show color?

-- 
Felipe Contreras

      reply	other threads:[~2021-06-28 21:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-08 17:49 How dow we educate our users to configure less? Felipe Contreras
2021-06-28 18:11 ` Felipe Contreras
2021-06-28 18:46   ` Randall S. Becker
2021-06-28 19:01     ` Felipe Contreras
2021-06-28 19:34       ` Randall S. Becker
2021-06-28 19:44         ` Felipe Contreras
2021-06-28 19:58           ` Randall S. Becker
2021-06-28 20:17             ` Felipe Contreras
2021-06-28 20:37               ` Randall S. Becker
2021-06-28 20:51                 ` Felipe Contreras
2021-06-28 20:56                   ` Randall S. Becker
2021-06-28 21:09                     ` Felipe Contreras [this message]

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=60da3a6e157de_1de4220874@natae.notmuch \
    --to=felipe.contreras@gmail.com \
    --cc=avarab@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=peff@peff.net \
    --cc=rsbecker@nexbridge.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.