On Thu, Aug 27, 2015 at 11:02 PM, Ganesh Ajjanagadde <gajja...@mit.edu> wrote: > On Thu, Aug 27, 2015 at 7:23 PM, Timothy Gu <timothyg...@gmail.com> wrote: >> On Thu, Aug 27, 2015 at 06:17:32PM -0400, Ganesh Ajjanagadde wrote: >>> This patch introduces color warning messages (yellow). >>> Tested with zsh, bash, and sh (bash under --posix). >>> >>> Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> >>> --- >>> configure | 4 +++- >>> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> I'm concerned that this would trash the configure output when it is >> viewed with something not supporting ANSI support codes. For example, >> when one pipes the output to a file. > > FWIW, config.log (by default) stores the configure output (including > the warnings). > Note that this patch does not affect the writes to logs, only the > stdout (which may be piped as you pointed out). > Thus this patch is not too bad in this aspect.
Please see new version which should not trash under piping. It also checks if the terminal supports colors in a POSIX compliant way. > >> >> Timothy >> _______________________________________________ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
