On Thu, Aug 27, 2015 at 11:33:52PM -0400, Ganesh Ajjanagadde wrote:
> This patch introduces color warning messages (yellow).
> They are enabled whenever available.
> Tested with zsh, bash, and sh (bash under --posix).
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com>
> ---
>  configure | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index 5913090..4f712c1 100755
> --- a/configure
> +++ b/configure
> @@ -416,6 +416,13 @@ EOF
>  }
>  
>  quotes='""'
> +if test -t 1; then
> +    ncolors=$(tput colors)
> +    if test -n "$ncolors" && test $ncolors -ge 8; then
> +        warn_color=$'\033[33;1m'
> +        reset_color=$'\033[0m'
> +    fi
> +fi
>  
>  log(){
>      echo "$@" >> $logfile
> @@ -6201,7 +6208,7 @@ echo "#endif /* AVUTIL_AVCONFIG_H */" >> $TMPH
>  cp_if_changed $TMPH libavutil/avconfig.h
>  
>  if test -n "$WARNINGS"; then
> -    printf "\n$WARNINGS"
> +    printf "\n%s$WARNINGS%s" "$warn_color" "$reset_color"
>      enabled fatal_warnings && exit 1
>  fi
>  

Aren't you supposed to use tput(1) for maximum compatibility?

-- 
Clément B.

Attachment: pgp0GlIwxBfZo.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to