[FFmpeg-devel] [PATCHv2] configure: colorize warning messages
Clément Bœsch
u at pkh.me
Fri Aug 28 14:36:57 CEST 2015
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 <gajjanagadde at 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150828/a7cafef2/attachment.sig>
More information about the ffmpeg-devel
mailing list