{"thread":{"id":"31963","subject":"[PATCH] Documentation: improve the example of overriding LESS via core.pager","startedAt":"2012-10-28T20:12:46Z","lastAt":"2012-10-29T06:26:57Z","messageCount":2,"participants":["Patrick Palka","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"202062","messageId":"1351455166-2579-1-git-send-email-patrick@parcs.ath.cx","threadId":"31963","inReplyTo":null,"subject":"[PATCH] Documentation: improve the example of overriding LESS via core.pager","fromName":"Patrick Palka","fromEmail":"patrick@parcs.ath.cx","sentAt":"2012-10-28T20:12:46Z","receivedAt":"2012-10-28T20:12:46Z","isPatch":true,"sender":{"key":"patrick@parcs.ath.cx","avatar":"https://avatars.githubusercontent.com/u/320253206?v=4"},"body":"You can override an option set in the LESS variable by simply prefixing\nthe command line option with `-+`. This is more robust than the previous\nexample if the default LESS options are to ever change.\n\nSigned-off-by: Patrick Palka <patrick@parcs.ath.cx>\n---\n Documentation/config.txt |    8 ++++----\n 1 file changed, 4 insertions(+), 4 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 11f320b..9a0544c 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -538,14 +538,14 @@ core.pager::\n \t`LESS` variable to some other value.  Alternately,\n \tthese settings can be overridden on a project or\n \tglobal basis by setting the `core.pager` option.\n-\tSetting `core.pager` has no affect on the `LESS`\n+\tSetting `core.pager` has no effect on the `LESS`\n \tenvironment variable behaviour above, so if you want\n \tto override git's default settings this way, you need\n \tto be explicit.  For example, to disable the S option\n \tin a backward compatible manner, set `core.pager`\n-\tto `less -+$LESS -FRX`.  This will be passed to the\n-\tshell by git, which will translate the final command to\n-\t`LESS=FRSX less -+FRSX -FRX`.\n+\tto `less -+S`.  This will be passed to the shell by\n+\tgit, which will translate the final command to\n+\t`LESS=FRSX less -+S`.\n \n core.whitespace::\n \tA comma separated list of common whitespace problems to\n-- \n1.7.10.4\n"},{"id":"202090","messageId":"20121029062657.GB5102@sigill.intra.peff.net","threadId":"31963","inReplyTo":"1351455166-2579-1-git-send-email-patrick@parcs.ath.cx","subject":"Re: [PATCH] Documentation: improve the example of overriding LESS via core.pager","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2012-10-29T06:26:57Z","receivedAt":"2012-10-29T06:26:57Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Oct 28, 2012 at 04:12:46PM -0400, Patrick Palka wrote:\n\n> You can override an option set in the LESS variable by simply prefixing\n> the command line option with `-+`. This is more robust than the previous\n> example if the default LESS options are to ever change.\n\nYeah, the current description is quite tortured and complex. I wondered\nif there might be some reason, but I think it is simply that fee7545 was\nover-zealous in adopting the original text. Yours is much more sensible.\n\nThanks.\n\n-Peff\n"}]}