[LTP] [PATCH 2/2] color: Fix backslash-escape sequences for some non-bash shells
Cyril Hrubis
chrubis@suse.cz
Wed Mar 1 15:03:13 CET 2017
Hi!
> > > # NOTE: these colors should match colors defined in include/tst_ansi_color.h
> > > - local ansi_color_blue='\e[1;34m'
> > > - local ansi_color_green='\e[1;32m'
> > > - local ansi_color_magenta='\e[1;35m'
> > > - local ansi_color_red='\e[1;31m'
> > > - local ansi_color_yellow='\e[1;33m'
> > > + local ansi_color_blue='\033[34;1m'
> > > + local ansi_color_green='\033[32;1m'
> > > + local ansi_color_magenta='\033[35;1m'
> > > + local ansi_color_red='\033[31;1m'
> > > + local ansi_color_yellow='\033[33;1m'
>
> > I'm a bit puzzled by the change of the position of color and bold in the
> > escape sequence. As far as I can tell it does not matter at all, or does
> > it?
> Sorry, my inattention, I'll generate v2.
I'm not saying that it's wrong. Just that there is no mention of the
change in the commit description and without it it looks meaningless.
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list