ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Slava Semushin" <slava.semushin@gmail.com>
To: "ALT Linux Team development discussions" <devel@lists.altlinux.org>
Subject: Re: [devel] Покритикуйте init-скрипт
Date: Wed, 31 Oct 2007 16:50:19 +0600
Message-ID: <75e139a00710310350p61fa09a7v447655666f3b0cc8@mail.gmail.com> (raw)
In-Reply-To: <4728588B.7090408@mail.ru>

31.10.07, Avramenko Andrew<liksx / mail.ru> написал(а):
> Сильно не вчитывался, но не понравился кусок:
>
> else
>         exit 1
> fi

Мда.. даже и не знаю что туда написать. Я думал, что-нить выдать, но
этот случай по идее не должен случиться. Это типа assert() такой.
Потому что в файле /etc/ossec-init.conf всегда содержится определение
переменной TYPE и оно может принимать только три известных значения.

Можно написать такое: "Error: wrong installation type in /etc/ossec-init.conf!"

Так лучше?

> Лично я предпочитаю когда скрипт ругнется из-за чего вышел. Это обычно
> сильно облегчает процесс отладки.

Ага, оно конечно круто.. :)

Я когда этот инитскрипт отлаживал столкнулся с проблемой: инит скрипт
тихо отрабатывал ничег не делая. И только sh -x и RTFS помогли понять,
что путь к демону должен быть абсолютным, иначе ф-ция start_daemon не
сможет его запустить. При этом сама ф-ция никакой ошибки не вывела...


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

  reply	other threads:[~2007-10-31 10:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-31 10:22 Slava Semushin
2007-10-31 10:27 ` Avramenko Andrew
2007-10-31 10:50   ` Slava Semushin [this message]
2007-10-31 11:16     ` Avramenko Andrew
2007-11-01 10:46     ` Vladimir V. Kamarzin
2007-11-01 11:02       ` Slava Semushin
2007-11-01 12:14         ` Dmitry V. Levin
2007-11-01 12:19           ` Slava Semushin
2007-11-01 12:28             ` Dmitry V. Levin
2007-11-01 12:36               ` Slava Semushin
2007-11-01 13:46                 ` Dmitry V. Levin
2007-11-02 10:17               ` Slava Semushin
2007-10-31 11:08 ` Alexey I. Froloff
2007-11-01 12:26 ` Dmitry V. Levin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=75e139a00710310350p61fa09a7v447655666f3b0cc8@mail.gmail.com \
    --to=slava.semushin@gmail.com \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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