Re: [PATCH v7] help: colorize man pages if man.color=true under less(1)

Jeff King [off-list ref] writes:

I'm still entirely unconvinced that this should be in Git at all;
pointing GIT_MAN_VIEWER or man.*.cmd at a color-man wrapper seems like
it would be sufficient. But it feels like that conversation was not
going anywhere productive; I mention it here only to indicate that my
response above is not an endorsement of the concept.

I have the same reaction to the patch.

Junio C Hamano wrote:

Jeff King [off-list ref] writes:

quoted

I'm still entirely unconvinced that this should be in Git at all;
pointing GIT_MAN_VIEWER or man.*.cmd at a color-man wrapper seems like
it would be sufficient. But it feels like that conversation was not
going anywhere productive; I mention it here only to indicate that my
response above is not an endorsement of the concept.

I have the same reaction to the patch.

Do you care to explain why you think so?

What is the alternative in your view? What configuration should users
use instead?

-- 
Felipe Contreras