[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] configure.ac: support -fdiagnostics-color=always
From: |
Lennart Sorensen |
Subject: |
Re: [PATCH] configure.ac: support -fdiagnostics-color=always |
Date: |
Tue, 7 Apr 2015 11:54:27 -0400 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Apr 06, 2015 at 09:24:26PM +0300, Andrei Borzenkov wrote:
> В Mon, 6 Apr 2015 19:33:30 +0200
> Michael Zimmermann <address@hidden> пишет:
>
> > ---
> > configure.ac | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/configure.ac b/configure.ac
> > index 891c14f..1f8cb2c 100644
> > --- a/configure.ac
> > +++ b/configure.ac
> > @@ -1046,6 +1046,15 @@ if test "x$grub_cv_target_cc_qn" = xyes; then
> > TARGET_CFLAGS="$TARGET_CFLAGS -Qn"
> > fi
> >
> > +AC_CACHE_CHECK([whether option -fdiagnostics-color=always works],
> > grub_cv_target_cc_fdiagnostics_color, [
> > + CFLAGS="$TARGET_CFLAGS -fdiagnostics-color=always -Werror"
> > + AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[]], [[]])],
> > + [grub_cv_target_cc_fdiagnostics_color=yes],
> > + [grub_cv_target_cc_fdiagnostics_color=no])])
> > +if test "x$grub_cv_target_cc_fdiagnostics_color" = xyes; then
> > + TARGET_CFLAGS="$TARGET_CFLAGS -fdiagnostics-color=always"
> > +fi
> > +
> > #
> > # Compiler features.
> > #
>
> I think it should be left to distribution/user. You can always run
>
> ./configure TARGET_CFLAGS="-Os -fdiagnostic-color=always"
Yes please. Having your text build logs corrupted by colour codes is
not helpful in any way. I didn't even know that awful option existed.
Now if it had a =auto like ls and grep and such have where if the output
is a terminal, it does colour, but otherwise it does not, well then at
least it doesn't bother people logging the output.
--
Len Sorensen