Re: [PATCH] Make -fdiagnostics-color= default configurable, default to =auto

[Jeff Law]

On 11/12/14 15:33, Jakub Jelinek wrote:

Hi!

This patch makes the -fdiagnostics-color= default configurable, and
changes the default (if no configure option is specified for it)
to --with-diagnostics-color=auto.  The previous behavior can be
restored with --with-diagnostics-color=auto-if-env , the 4.8
behavior (never coloring anything) with --with-diagnostics-color=never .
Bootstrapped/regtested on x86_64-linux and i686-linux and tested with
all 5 different configure options (the four explicit one and without).
Ok for trunk?

2014-11-12  Jakub Jelinek  <ja...@redhat.com>

        * configure.ac (--with-diagnostics-color): New configure
        option, default to --with-diagnostics-color=auto.
        * toplev.c (process_options): Use DIAGNOSTICS_COLOR_DEFAULT
        to determine -fdiagnostics-color= option default.
        * doc/invoke.texi (-fdiagnostics-color=): Document new
        default.
        * configure: Regenerated.
        * config.in: Regenerated.

OK.  Should the change in default behaviour be mentioned somewhere?

Jeff