* 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