On Jan 20, 2014, at 21:30, Jeff King wrote:
Ugh. Having just read the LESS discussion, it makes me wonder if the

 strchr(getenv("LESS"), 'R')

check I add elsewhere in the series is sufficient. I suspect in practice
it is good enough, but I would not be surprised if there is a way to
fool it with a sufficiently complex LESS variable. Maybe we should just
assume it is enough, and people with crazy LESS settings can set
color.pager as appropriate?

This attempts to show colors but doesn't work properly:

LESS=-+R git -c color.ui=auto -c color.pager=true log --oneline -- graph

but this does

LESS=-R git -c color.ui=auto -c color.pager=true log --oneline -- graph

so yeah, just checking for 'R' in $LESS is only approximate.  :)
Also -r is good enough to show colors too...
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to