On Wed, Aug 15, 2018 at 10:24:31AM -0600, David Ahern wrote: > On 8/15/18 10:21 AM, Phil Sutter wrote: > > Add an additional prerequisite to check_enable_color() to make sure > > stdout actually points to an open TTY device. Otherwise calls like > > > > | ip -color a s >/tmp/foo > > > > will print color escape sequences into that file. Allow to override this > > check by specifying '-color' flag more than once. > > > > Signed-off-by: Phil Sutter <p...@nwl.cc> > > --- > > Changes since v1: > > - Allow to override isatty() check by specifying '-color' flag more than > > once. > > That adds overhead to my workflow where I almost always have to pipe the > output of ip to a pager.
alias ip='ip -color -color' Another alternative may be to introduce -autocolor flag. Establishing the same syntax as used by 'ls' is not as trivial due to the simple commandline parsing used in 'ip'. Cheers, Phil
