[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