Hello,

the "bash shell startup files" section can be improved:

1) /etc/profile tests for EUID=0 in two different ways:

|if [ $EUID -eq 0 ] ; then
        pathappend /sbin:/usr/sbin
        unset HISTFILE
fi|


|if [[ $EUID == 0 ]] ; then
  PS1="$RED\u [ $NORMAL\w$RED ]# $NORMAL"
else
  PS1="$GREEN\u [ $NORMAL\w$GREEN ]\$ $NORMAL"
fi

|

Please remove the "==" construction, it is completely unneeded here, 
"-eq" works just fine. Also, "#" can be replced with "\$", too.

2) /etc/profile.d/extrapaths.sh uses the [ -d /usr/local/sbin -a $EUID 
-eq 0 ] construction, but the "-a" doesn't work in all shells (try 
posh). Suggestion: [ -d /usr/local/sbin ] && [ $EUID -eq 0 ]

3) /etc/profile.d/readline.sh is completely unneeded - readline-5.2 
looks in /etc/inputrc by default.

4) It should be mentioned that umask (as set in the book) doesn't work 
for non-shell logins (e.g., scp or svn-over-ssh). A recommendation to 
use pam-umask 
(http://ftp.debian.org/debian/pool/main/p/pam-umask/pam-umask_0.04.tar.gz) 
may be more suitable.||

-- 
Alexander E. Patrakov
-- 
http://linuxfromscratch.org/mailman/listinfo/blfs-dev
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to