* Re: [devel] [SCM] packages/avreg: heads/master
@ 2008-07-04 5:08 ` Slava Semushin
2008-07-04 9:30 ` Denis Klimov
0 siblings, 1 reply; 5+ messages in thread
From: Slava Semushin @ 2008-07-04 5:08 UTC (permalink / raw)
To: ALT Devel discussion list
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [SCM] packages/avreg: heads/master
2008-07-04 5:08 ` [devel] [SCM] packages/avreg: heads/master Slava Semushin
@ 2008-07-04 9:30 ` Denis Klimov
2008-07-05 21:13 ` [devel] [JT] shell scripting Michael Shigorin
0 siblings, 1 reply; 5+ messages in thread
From: Denis Klimov @ 2008-07-04 9:30 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [devel] [JT] shell scripting
2008-07-04 9:30 ` Denis Klimov
@ 2008-07-05 21:13 ` Michael Shigorin
2008-07-05 21:19 ` Slava Semushin
0 siblings, 1 reply; 5+ messages in thread
From: Michael Shigorin @ 2008-07-05 21:13 UTC (permalink / raw)
To: ALT Linux Team development discussions
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/
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [JT] shell scripting
2008-07-05 21:13 ` [devel] [JT] shell scripting Michael Shigorin
@ 2008-07-05 21:19 ` Slava Semushin
2008-07-19 11:49 ` Stanislav Ievlev
0 siblings, 1 reply; 5+ messages in thread
From: Slava Semushin @ 2008-07-05 21:19 UTC (permalink / raw)
To: ALT Linux Team development discussions
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [devel] [JT] shell scripting
2008-07-05 21:19 ` Slava Semushin
@ 2008-07-19 11:49 ` Stanislav Ievlev
0 siblings, 0 replies; 5+ messages in thread
From: Stanislav Ievlev @ 2008-07-19 11:49 UTC (permalink / raw)
To: ALT Linux Team development discussions
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@ вроде бы не пишет в подобном стиле.
Зато ошибается чаще ;)
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2008-07-19 11:49 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-07-04 5:08 ` [devel] [SCM] packages/avreg: heads/master Slava Semushin
2008-07-04 9:30 ` Denis Klimov
2008-07-05 21:13 ` [devel] [JT] shell scripting Michael Shigorin
2008-07-05 21:19 ` Slava Semushin
2008-07-19 11:49 ` Stanislav Ievlev
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git