From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zver@distance.ru>
X-Spam-Checker-Version: SpamAssassin 3.2.4 (2008-01-01) on sa.int.altlinux.org
X-Spam-Level: 
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00 autolearn=ham
	version=3.2.4
Date: Fri, 4 Jul 2008 15:30:26 +0600
From: Denis Klimov <zver@distance.ru>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Message-Id: <20080704153026.1aed2e2f.zver@distance.ru>
In-Reply-To: <75e139a00807032208k17a77352i842b1a6dc3ef4399@mail.gmail.com>
References: <20080704043540.79573164003D@ssh.git.int.altlinux.org>
	<75e139a00807032208k17a77352i842b1a6dc3ef4399@mail.gmail.com>
Organization: IDO
X-Mailer: Sylpheed version 2.2.9 (GTK+ 2.12.8; i586-alt-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=KOI8-R
Content-Transfer-Encoding: 8bit
X-Virus-Scanned: ClamAV using ClamSMTP
Subject: Re: [devel] [SCM] packages/avreg: heads/master
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.10b3
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Fri, 04 Jul 2008 09:30:20 -0000
Archived-At: <http://lore.altlinux.org/devel/20080704153026.1aed2e2f.zver@distance.ru/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

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