[Comm] Re: Покритикуйте spec

[Michael Shigorin]

On Wed, Feb 16, 2005 at 01:59:01PM +0200, Slava Dubrovskiy wrote:
> 1. Как выбрать приоритеты для start/stop?

По вкусу, сообразуясь с требованиями (сеть, база, syslog, ...).

> 2. Что лучше, сделать исправление в спеке чтобы он добавлял
> необходимые строки в файл сервиса или делать Patch?

Патч обычно удобней.  Впрочем, инитскрипт обычно проще переписать
с нуля (точнее, с /etc/init.d/template).

> Может еще что-то не правильно?

> Summary: Mini-ATS loger

logger :)

> %define atsloginit atslog
> 
> %prep
> %setup
> cd %_builddir/%name-%version/include
> %__subst "s/\/usr\/local\/etc/\/etc/g" atslogcleardb.pl atslogd atslogdaily atslogdb.pl atslogmaster atslogrotate

Может быть удобней так:

%__subst "s,/usr/local/etc/,/etc,g" atslogcleardb.pl atslogd atslogdaily atslogdb.pl atslogmaster atslogrotate

или так:

%__subst "s|/usr/local/etc/|/etc|g" atslogcleardb.pl atslogd atslogdaily atslogdb.pl atslogmaster atslogrotate

> %__subst "s/\/usr\/local//g" atslogdinit.asplinux

Это не будет соответствовать текущей полиси по инитскриптам --
они должны применять start-stop-service, а не RH functions.

> %__install -pD -m 755 src/atslogcat/atslogcat %buildroot%_bindir/atslogcat

Для краткости можно

%__install -pD -m755 src/atslogcat/atslogcat %buildroot%_bindir/

(при уже созданном %buildroot%_bindir/ -- я обычно такие каталоги
создаю пачкой заранее, а то установку файлов читать трудно)

> %doc CHANGES DEINSTALL INSTALL README UPDATING USING src/atslogcat/readme_eng.txt src/atslogcat/readme_rus.txt include/atslog.conf.default

Можно разбить:

%doc CHANGES DEINSTALL INSTALL README UPDATING USING
%doc src/atslogcat/readme_eng.txt src/atslogcat/readme_rus.txt
%doc include/atslog.conf.default

-- 
 ---- WBR, Michael Shigorin <mike на altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/