bug in /etc/profile.d/colorls.sh (fileutils-4.0.36-4 and older)

[Brent Thompson]

colorls.sh sets aliases for ls(1) depending on situation.  It queries
/etc/DIR_COLORS to determine whether color is set to disabled, but it
doesn't check that file for whether $TERM is a terminal that supports color
or not, hence sets inappropriate aliases if COLOR is enabled but TERM does
not support color.  Shouldn't colorls.sh be changed to be this:

# diff colorls.sh.orig colorls.sh.new
7,8c7
<   if ! egrep -qi "^COLOR.*none" $COLORS &>/dev/null && \
<    egrep -qi "^TERM[  ]*$TERM" $COLORS &>/dev/null; then
---
>   if ! egrep -qi "^COLOR.*none" $COLORS &>/dev/null; then


 ---   Brent