[FFmpeg-devel] [PATCH] configure: colorize warning messages

[Ganesh Ajjanagadde]

On Thu, Aug 27, 2015 at 11:02 PM, Ganesh Ajjanagadde <gajjanag at mit.edu> wrote:
> On Thu, Aug 27, 2015 at 7:23 PM, Timothy Gu <timothygu99 at gmail.com> wrote:
>> On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote:
>>> This patch introduces color warning messages (yellow).
>>> Tested with zsh, bash, and sh (bash under --posix).
>>>
>>> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
>>> ---
>>>  configure | 4 +++-
>>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> I'm concerned that this would trash the configure output when it is
>> viewed with something not supporting ANSI support codes. For example,
>> when one pipes the output to a file.
>
> FWIW, config.log (by default) stores the configure output (including
> the warnings).
> Note that this patch does not affect the writes to logs, only the
> stdout (which may be piped as you pointed out).
> Thus this patch is not too bad in this aspect.

Please see new version which should not trash under piping.
It also checks if the terminal supports colors in a POSIX compliant way.

>
>>
>> Timothy
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel