Re: [devel] [git update] packages/katrin: heads/master

Re: [devel] [git update] packages/katrin: heads/master

[Slava Semushin]

2007/7/4, Denis Klimov <zver / altlinux.org>:
[...]
> +install -pD -m 644 src/init.d/%name %buildroot/etc/init.d/%name

-install -pD -m 644 src/init.d/%name %buildroot/etc/init.d/%name
+install -pD -m 644 src/init.d/%name %buildroot%_initdir/%name

Предлагаю вместо /etc/init.d использовать макрос %_initdir. Правда, он
раскрывается в /etc/rc.d/init.d

*глянул в спек ldv@ для bind*

Думаю, что правильнее использовать этот макрос и всё-таки хранить
скрипт для запуска сервиса в этом каталоге :)

>  %files
>  %_bindir/*
>  %_libdir/%name

Вы забыли добавить скрипт для запуска сервиса в пакет:

+%_initdir/%name

[...]
>  %changelog
> +* Wed Jul 04 2007 Denis Klimov <zver@altlinux.ru> 0.4.10-alt1
> +- add %post and %preun
> +- install init script

-- add %post and %preun
+- add %%post and %%preun

[...]
> diff --git a/src/init.d/katrin b/src/init.d/katrin
> new file mode 100644
> index 0000000..3c9a664
> --- /dev/null
> +++ b/src/init.d/katrin
[...]
> +BINARY=/usr/bin/katrin
> +LOCKFILE=/var/lock/subsys/hddtemp
> +SEPARATOR="|"
> +RETVAL=0
[...]

Переменная SEPARATOR не используется в скрипте. Она точно нужна?


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

Re: [devel] [git update] packages/katrin: heads/master

[Epiphanov Sergei]

В сообщении от Thursday 05 July 2007 10:25:38 Slava Semushin написал(а):
> Предлагаю вместо /etc/init.d использовать макрос %_initdir. Правда, он
> раскрывается в /etc/rc.d/init.d

В реальности /etc/init.d у нас - это символьная ссылка на /etc/rc.d/init.d. И 
правильнее пользоваться именно последним.

-- 
С уважением, Епифанов Сергей

Re: [devel] [git update] packages/katrin: heads/master

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 522 bytes --]

On Thu, Jul 05, 2007 at 10:58:09AM +0400, Epiphanov Sergei wrote:
> В сообщении от Thursday 05 July 2007 10:25:38 Slava Semushin написал(а):
> > Предлагаю вместо /etc/init.d использовать макрос %_initdir. Правда, он
> > раскрывается в /etc/rc.d/init.d
> 
> В реальности /etc/init.d у нас - это символьная ссылка на /etc/rc.d/init.d. И 
> правильнее пользоваться именно последним.

Это очень тонкий вопрос, каким из этих двух имён правильнее пользоваться.
В spec-файле лучше использовать %_initdir.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [git update] packages/katrin: heads/master

[Denis Klimov]

On Thu, 5 Jul 2007 13:25:38 +0700 Slava Semushin wrote:


> 
> Переменная SEPARATOR не используется в скрипте. Она точно нужна?
> 
Пока нет, я просто взял за основу другой init скрипт.

--
Denis Klimov 
zver

Re: [devel] [git update] packages/katrin: heads/master

[Epiphanov Sergei]

В сообщении от Friday 06 July 2007 11:19:31 Dmitry V. Levin написал(а):
> > В реальности /etc/init.d у нас - это символьная ссылка на
> > /etc/rc.d/init.d. И правильнее пользоваться именно последним.
>
> Это очень тонкий вопрос, каким из этих двух имён правильнее пользоваться.
> В spec-файле лучше использовать %_initdir.

Я понимаю, что тонкий. Но если судить по RedHat (насколько помню по версии 
5.0), то там /etc/init.d сначала вообще отсутствовал и появился позднее.

-- 
С уважением, Епифанов Сергей