git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2] pager: remove 'S' from $LESS by default

From
Junio C Hamano <gitster@pobox.com>
Date
May 7, 2014, 17:07 UTC
Message-ID
<xmqqy4ydbjqm.fsf@gitster.dls.corp.google.com>
In-Reply-To
<20140506215516.GA30185@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 11 quoted lines
> On Tue, May 06, 2014 at 08:49:22PM +0200, Matthieu Moy wrote:
> ...
>> The idea of having a separate default value for pager.blame (or set
>> $LESS differently for blame) crossed my mind, but I actually don't like
>> it, as it would make it harder for a user to fine-tune his configuration
>> manually (one would have to cancel all the corner-cases that Git would
>> set by default).
>
> Agreed. We already get some confusion from users with "git has set $LESS
> for me".  Changing it to "git set up $LESS depending on which command is
> running" seems like it would cause more of the same.

While I fully agree with the above conclusion, I just noticed that I will be irritated enough to eventually set pager.blame myself, after running a short "git blame -L1311,+7 git-p4.py", which is one of the standard first steps for me to start reading patches submit on the list.

Even with line wrapping, the output fits on a single page, so "F" (aka "--quit-if-one-screen") kicks in, before giving me a chance to scroll horizontally around.

Not objecting to the conclusion of the discussion with a concrete counterproposal. Just hoping somebody clever enough might come up with a good trick to make things easier to use and explain, which I however suspect may be an incompatible pair of goals.

Previous: Jeff KingNext: Matthieu Moy
Message 39 of 41 in “Harmful LESS flags”
  1. d9ba@mailtor.netApr 23, 2014
  2. Jonathan NiederApr 24, 2014
  3. Mark NudelmanApr 28, 2014
  4. David KastrupApr 24, 2014
  5. Junio C HamanoApr 24, 2014
  6. David KastrupApr 24, 2014
  7. Junio C HamanoApr 24, 2014
  8. David KastrupApr 24, 2014
  9. Jeff KingApr 24, 2014
  10. Junio C HamanoApr 24, 2014
  11. Jeff KingApr 24, 2014
  12. David KastrupApr 24, 2014
  13. Jeff KingApr 24, 2014
  14. David KastrupApr 24, 2014
  15. Jonathan NiederApr 24, 2014
  16. Matthieu MoyApr 25, 2014
  17. Jonathan NiederApr 25, 2014
  18. David KastrupApr 25, 2014
  19. Jonathan NiederApr 25, 2014
  20. PAGER_ENV: remove 'S' from $LESS by defaultMatthieu Moy, Apr 28, 2014
  21. David KastrupApr 28, 2014
  22. Matthieu MoyApr 28, 2014
  23. David KastrupApr 28, 2014
  24. Matthieu MoyApr 28, 2014
  25. Jeff KingApr 28, 2014
  26. Junio C HamanoApr 28, 2014
  27. Matthieu MoyApr 29, 2014
  28. Junio C HamanoApr 29, 2014
  29. pager: remove 'S' from $LESS by defaultMatthieu Moy, Apr 30, 2014
  30. Junio C HamanoApr 30, 2014
  31. Matthieu MoyApr 30, 2014
  32. Junio C HamanoApr 30, 2014
  33. Jonathan NiederMay 5, 2014
  34. Matthieu MoyMay 5, 2014
  35. Junio C HamanoMay 6, 2014
  36. David KastrupMay 6, 2014
  37. Matthieu MoyMay 6, 2014
  38. Jeff KingMay 6, 2014
  39. Junio C HamanoMay 7, 2014
  40. Matthieu MoyMay 7, 2014
  41. Junio C HamanoMay 7, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.