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

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

[Slava Semushin]

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, ведь если переменная окажется пустой, то условие будет
некорректным и вы получите синтаксическую ошибку.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

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

[Denis Klimov]

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

[devel] [JT] shell scripting

[Michael Shigorin]

On Fri, Jul 04, 2008 at 03:30:26PM +0600, Denis Klimov wrote:
> > И далее везде -- заключайте переменные в двойные кавычки,
> > особенно в условиях if, ведь если переменная окажется пустой,
> > то условие будет некорректным и вы получите синтаксическую
> > ошибку.
> Благодарю за советы и внимание.

Этот был по делу :)

> В программировании на shell я пока новичок.

Сильно рекомендую пакеты ldv@, legion@, inger@, at@
(e.g. control, libshell...), если найдёте -- Керниган, Пайк
"UNIX: универсальная среда программирования" (М.: Финансы и
статистика, 1992. - 304 с.: ил., ISBN 5-279-00253-4), а также
http://gazette.linux.ru.net/rus/articles/abs-guide/index.html

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [JT] shell scripting

[Slava Semushin]

6 июля 2008 г. 4:13 пользователь Michael Shigorin <mike / osdn.org.ua> написал:
>> В программировании на shell я пока новичок.
> Сильно рекомендую пакеты ldv@, legion@, inger@, at@

Есть мнение, что рекомендовать изучать их исходники новичкам -- не
самая лучшая идея.


P.S. Отвечая на вопрос "почему?" отвечаю -- зачастую там читаемость на
на высшем уровне и доступна только тем кто может легко вопринимать set
-e код и на память знает все короткие ${} конструкции.

P.P.S. Кстати, inger@ вроде бы не пишет в подобном стиле.

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] [JT] shell scripting

[Stanislav Ievlev]

On Sun, Jul 06, 2008 at 04:19:33AM +0700, Slava Semushin wrote:
> 6 июля 2008 г. 4:13 пользователь Michael Shigorin <mike / osdn.org.ua> написал:
> >> В программировании на shell я пока новичок.
> > Сильно рекомендую пакеты ldv@, legion@, inger@, at@
> 
> Есть мнение, что рекомендовать изучать их исходники новичкам -- не
> самая лучшая идея.
> 
> 
> P.S. Отвечая на вопрос "почему?" отвечаю -- зачастую там читаемость на
> на высшем уровне и доступна только тем кто может легко вопринимать set
> -e код и на память знает все короткие ${} конструкции.
> 
> P.P.S. Кстати, inger@ вроде бы не пишет в подобном стиле.
Зато ошибается чаще ;)