From: Felipe Contreras <felipe.contreras@gmail.com>
Date: Thu, 24 Jun 2021 01:44:57 GMT
Subject: RE: [PATCH v6] help: colorize man pages
Message-ID: <60d3e399945d_2cc520855@natae.notmuch>
In-Reply-To: <20210523054454.1188757-1-felipe.contreras@gmail.com>

> --- a/builtin/help.c
> +++ b/builtin/help.c

> @@ -264,6 +289,7 @@ static void exec_man_man(const char *path, const char *page)
>  static void exec_man_cmd(const char *cmd, const char *page)
>  {
>  	struct strbuf shell_cmd = STRBUF_INIT;
> +	colorize_man();

On further reflection I don't think this colorize belongs here.
exec_man_cmd() is meant to execute any custom command, not necessarily
man.

-- 
Felipe Contreras