[FFmpeg-devel] Allow to force colored output
Etienne Buira
etienne.buira.lists
Sun Nov 14 17:09:31 CET 2010
On Sun, Nov 14, 2010 at 03:47:06PM +0100, Stefano Sabatini wrote:
> On date Sunday 2010-11-14 15:02:58 +0100, Etienne Buira encoded:
> > On Sun, Nov 14, 2010 at 02:44:02PM +0100, Etienne Buira wrote:
> > > On Sun, Nov 14, 2010 at 02:23:29PM +0100, Stefano Sabatini wrote:
> [...]
> > > > We should provide a namespace even in this case, I suggest:
> > > > FFMPEG_NO_COLOR
> > > > FFMPEG_FORCE_COLOR
> > >
> > > Done, but might break for people who use NO_COLOR yet.
> > >
> > > > Also can you say why do you need this?
> > >
> > > My usecase is to save ffmpeg's output (ffmpeg ... > log 2>&1), but still
> > > having nice colors.
> >
> > Sorry for the noise, this one should actually not screw up things for
> > win32.
>
> > Index: libavutil/log.c
> > ===================================================================
> > --- libavutil/log.c (revision 25751)
> > +++ libavutil/log.c (working copy)
> > @@ -55,16 +55,16 @@
> > #if defined(_WIN32) && !defined(__MINGW32CE__)
> > CONSOLE_SCREEN_BUFFER_INFO con_info;
> > con = GetStdHandle(STD_ERROR_HANDLE);
> > - use_color = (con != INVALID_HANDLE_VALUE) && !getenv("NO_COLOR");
> > + use_color = (con != INVALID_HANDLE_VALUE) && (!getenv("FFMPEG_NO_COLOR") || getenv("FFMPEG_FORCE_COLOR"));
> > if (use_color) {
> > GetConsoleScreenBufferInfo(con, &con_info);
> > attr_orig = con_info.wAttributes;
> > background = attr_orig & 0xF0;
> > }
> > #elif HAVE_ISATTY
> > - use_color= getenv("TERM") && !getenv("NO_COLOR") && isatty(2);
> > + use_color= (getenv("TERM") && !getenv("FFMPEG_NO_COLOR") && isatty(2)) || getenv("FFMPEG_FORCE_COLOR");
> > #else
> > - use_color= 0;
> > + use_color= !!getenv("FFMPEG_FORCE_COLOR");
> > #endif
> > }
>
> Don't mix separate changes, simply call the new variable
> FFMPEG_FORCE_COLOR, the change NO_COLOR -> FFMPEG_NO_COLOR belongs to
> a different patch.
Hope this one is ok, feel free to adapt to your taste.
-------------- next part --------------
Index: libavutil/log.c
===================================================================
--- libavutil/log.c (revision 25751)
+++ libavutil/log.c (working copy)
@@ -55,16 +55,16 @@
#if defined(_WIN32) && !defined(__MINGW32CE__)
CONSOLE_SCREEN_BUFFER_INFO con_info;
con = GetStdHandle(STD_ERROR_HANDLE);
- use_color = (con != INVALID_HANDLE_VALUE) && !getenv("NO_COLOR");
+ use_color = (con != INVALID_HANDLE_VALUE) && (!getenv("NO_COLOR") || getenv("FFMPEG_FORCE_COLOR"));
if (use_color) {
GetConsoleScreenBufferInfo(con, &con_info);
attr_orig = con_info.wAttributes;
background = attr_orig & 0xF0;
}
#elif HAVE_ISATTY
- use_color= getenv("TERM") && !getenv("NO_COLOR") && isatty(2);
+ use_color= (getenv("TERM") && !getenv("NO_COLOR") && isatty(2)) || getenv("FFMPEG_FORCE_COLOR");
#else
- use_color= 0;
+ use_color= !!getenv("FFMPEG_FORCE_COLOR");
#endif
}
Index: doc/fftools-common-opts.texi
===================================================================
--- doc/fftools-common-opts.texi (revision 25751)
+++ doc/fftools-common-opts.texi (working copy)
@@ -84,6 +84,7 @@
By default the program logs to stderr, if coloring is supported by the
terminal, colors are used to mark errors and warnings. Log coloring
-can be disabled setting the environment variable @env{NO_COLOR}.
+can be disabled setting the environment variable @env{NO_COLOR}, or can
+be forced setting the environment variable @env{FFMPEG_FORCE_COLOR}.
@end table
More information about the ffmpeg-devel
mailing list