Re: [devel] [SCM] packages/avreg: heads/master

[Denis Klimov <zver@distance.ru>]

On Fri, 4 Jul 2008 12:08:34 +0700 Slava Semushin wrote:

> 2008/7/4 Denis Klimov <zver / altlinux.org>:
> > Update of /people/zver/packages/avreg.git
> [...]
> > +++ b/common/create_db
> > @@ -100,7 +100,10 @@ feel_db() {
> >                echo "Database $AVREGDB already exist"
> >        fi
> >  }
> > -
> > +if [ $USER != 'root' ]; then
> > +       echo 'This script must be run by root'
> > +       exit 1
> > +fi
> >
> >  if [ -r $MYSQL_CNF ]; then
> >        chmod 0600 $MYSQL_CNF
> 
> Предлагаю хотя бы стремиться писАть более-менее стабильный код,
> который был бы устойчив к пробелам в именах файлах и пустым
> значениям... Этот код никуда не годится, вот так лучше, на мой взгляд:
> 
> if [ "$USER" != 'root' ]; then
>        echo 'This script must be run by root' >&2
>        exit 1
> fi
> 
> И далее везде -- заключайте переменные в двойные кавычки, особенно в
> условиях if, ведь если переменная окажется пустой, то условие будет
> некорректным и вы получите синтаксическую ошибку.
> 
Благодарю за советы и внимание. В программировании на shell я пока новичок.

--
Denis Klimov 
zver