Re: [PATCH] configure.ac: support -fdiagnostics-color=always

[Andrei Borzenkov <arvidjaar@gmail.com>]

В Mon, 6 Apr 2015 19:33:30 +0200
Michael Zimmermann <sigmaepsilon92@gmail.com> пишет:

> ---
>  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"