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

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

[Slava Dubrovskiy]

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

Здравствуйте.
Вот пытаюсь опакетить одну программу. Написал spec. Есть несколько вопросов.
Программа устанавливает сервис, а он не поддерживает chkconfig. Т.е. 
надо добавить что-то типа
       # chkconfig: 2345 20 80
       # description: Saves and restores system entropy pool for \
       #              higher quality random number generation.

Вопрос:
1. Как выбрать приоритеты для start/stop?
2. Что лучше, сделать исправление в спеке чтобы он добавлял необходимые 
строки в файл сервиса или делать Patch?
Может еще что-то не правильно?

-- 
С уважением,
Дубровский Вячеслав.


[-- Attachment #2: atslog.spec --]
[-- Type: text/plain, Size: 2975 bytes --]

Name: atslog
Version: 1.4
Release: alt0
Summary: Mini-ATS loger
License: GPL
Group: System/Servers
Url: http://www.atslog.dp.ua/
Packager: Slava Dubrovskiy <slava@elan.com.ua>
Source: http://www.atslog.dp.ua/download/%name-%version.tar.gz
Source1: atslog.conf
Requires: perl-DBD-mysql

%define atsloginit atslog

%description
ðÒÏÇÒÁÍÍÁ-ÄÅÍÏÎ, ÄÌÑ ÚÁÐÉÓÉ Ú×ÏÎËÏ× ÒÁÚÌÉÞÎÙÈ ÍÏÄÅÌÅÊ ÍÉÎÉ-áôó × ÂÁÚÕ ÄÁÎÎÙÈ MySQL.
äÌÑ ÁÎÁÌÉÚÁ ÉÓÐÏÌØÚÕÊÔÅ ÐÁËÅÔ atslog-web Ó ÄÏÓÔÕÐÏÍ Ë ÂÁÚÅ ÄÁÎÎÙÈ ÞÅÒÅÚ web-ÂÒÁÕÚÅÒ.

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

cd %_builddir/%name-%version/Linux
%__subst "s/\/usr\/local//g" atslogdinit.asplinux
%__subst "s/atslogdinit.sh/%atsloginit/g" atslogdinit.asplinux

%build
cd %_builddir/%name-%version/src/atslogdaemon
%_make_bin
cd %_builddir/%name-%version/src/atslogcat
%_make_bin
cd %_builddir/%name-%version

%install
%__mkdir_p %buildroot%_docdir/%name-%version
%__mkdir_p %buildroot%_logdir/%name

# install bin
%__install -pD -m 755 src/atslogcat/atslogcat %buildroot%_bindir/atslogcat
%__install -pD -m 755 src/atslogdaemon/atslogdaemon %buildroot%_bindir/atslogdaemon
%__install -pD -m 644 include/atslogd %buildroot%_bindir/atslogd
%__install -pD -m 755 include/atslogmaster %buildroot%_bindir/atslogmaster
%__install -pD -m 644 include/atslogcleardb.pl %buildroot%_bindir/atslogcleardb.pl
%__install -pD -m 644 include/atslogdb.pl %buildroot%_bindir/atslogdb.pl

# install lib
%__mkdir_p %buildroot%_libexecdir/%name
%__install -p -m 644 libexec/[^A-Z]* %buildroot%_libexecdir/%name

# install data
#%%__mkdir_p %buildroot%_datadir/%name
%__install -pD -m 644 include/createsqltables.sql.default %buildroot%_datadir/%name/createsqltables.sql.default
%__install -p -m 644 textlogs/* %buildroot%_datadir/%name

# install lang
%__mkdir_p %buildroot%_datadir/%name/lang
%__install -p -m 644 lang/* %buildroot%_datadir/%name/lang

# install configs
%__install -pD -m 644 %SOURCE1 %buildroot%_sysconfdir/atslog.conf
%__install -pD -T -m 755 Linux/atslogdinit.asplinux %buildroot%_initdir/%atsloginit

# install cron
%__install -pD -m 755 include/atslogrotate %buildroot%_sysconfdir/logrotate.d/atslogrotate
%__install -pD -m 755 include/atslogdaily %buildroot%_sysconfdir/cron.daily/atslogdaily

%post
%post_service %atsloginit

%preun
%preun_service %atsloginit

%files
%_bindir
%_libexecdir/%name
%config(noreplace) %_sysconfdir/%name.conf
%config %_sysconfdir/logrotate.d
%config %_sysconfdir/rc.d
%config %_sysconfdir/cron.daily

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

%attr(3770,root,root) %_logdir/%name

%changelog
* Wed Feb 16 2005 Slava Dubrovskiy <slava@elan.com.ua> 1.4-alt0
- Built for Master 2.4

[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/

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

[php-coder]

--- Slava Dubrovskiy 16.02.2005 17:59:01
+++ php-coder 16.02.2005 19:38:41

>%setup
>cd %_builddir/%name-%version/include

    AFAIK макрос %setup сам перейдет в %_builddir/%name-%version
    Т.е. можно написать просто cd include (кстати, разве для cd нет
    макроса?)

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

    Тут, как уже сказал Михаил Шигорин, лучше использовать другой
    разделитель -- для удобочитаемости.

-- 
 + php-coder
 + php-coder[at]ngs[dot]ru
 + ALT Linux Master 2.4

если вы не решили какой дистрибутив выбрать подпишитесь на
русскоязычную рассылку каждого и через неделю-месяц вы будете
очень хорошо ориентироваться
                    -- anonymous (linux.org.ru)

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

[Slava Dubrovskiy]

Спасибо Михаил, исправлю.
А необходимо ли как-то проверять наличие pid файла перед установкой и 
если он есть убивать?

-- 
С уважением,
Дубровский Вячеслав.

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

[Slava Dubrovskiy]

Michael Shigorin пишет:

>Патч обычно удобней.  Впрочем, инитскрипт обычно проще переписать
>с нуля (точнее, с /etc/init.d/template).
>  
>
Еще хочу спросить. Инитскрипт проверяет файл конфигурации, берет от туда 
настройки по языку и выдает сообщения в лог на том языке который в 
локали или если нет таких настроек, то на английском. Правильно я 
понимаю, что это не имеет смысла, т.к. скрипт запускается от root  и 
локаль все равно будет английской?

-- 
С уважением,
Дубровский Вячеслав.

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

[Vasya Makarov]

Slava Dubrovskiy пишет:
> Michael Shigorin пишет:
> 
>> Патч обычно удобней.  Впрочем, инитскрипт обычно проще переписать
>> с нуля (точнее, с /etc/init.d/template).
>>  
>>
> Еще хочу спросить. Инитскрипт проверяет файл конфигурации, берет от туда 
> настройки по языку и выдает сообщения в лог на том языке который в 
> локали или если нет таких настроек, то на английском. Правильно я 
> понимаю, что это не имеет смысла, т.к. скрипт запускается от root  и 
> локаль все равно будет английской?
> 
У рута, всё таки, может быть другая локаль. Видал я таких рутов :)

-- 
Regards,
Vasya

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

[Вячеслав Диконов]

В Срд, 16/02/2005 в 16:05 +0200, Slava Dubrovskiy пишет:
> Еще хочу спросить. Инитскрипт проверяет файл конфигурации, берет от туда 
> настройки по языку и выдает сообщения в лог на том языке который в 
> локали или если нет таких настроек, то на английском. Правильно я 
> понимаю, что это не имеет смысла, т.к. скрипт запускается от root  и 
> локаль все равно будет английской?
Имеет, потому что не у всех root английский. 
//Пуристы заткнитесь!//
-- 
Вячеслав Диконов <linuxbox@degunino.net>

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

[Michael Shigorin]

On Wed, Feb 16, 2005 at 03:47:58PM +0200, Slava Dubrovskiy wrote:
> А необходимо ли как-то проверять наличие pid файла перед
> установкой и если он есть убивать?

Если его существование не мешает запуску (что в некоторых случаях
бывает не так) -- то нехай собi живе :)

On Wed, Feb 16, 2005 at 04:05:56PM +0200, Slava Dubrovskiy wrote:
> Еще хочу спросить. Инитскрипт проверяет файл конфигурации,
> берет от туда настройки по языку и выдает сообщения в лог на
> том языке который в локали или если нет таких настроек, то на
> английском. Правильно я понимаю, что это не имеет смысла, т.к.
> скрипт запускается от root  и локаль все равно будет
> английской?

Эээ... ну это какой-то заумный инитскрипт, если Вам не влом это
портировать -- то, конечно, пожалуйста, но я бы забил. :)

// Трудоголики за других, молчать!!! //

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