[devel] Замечания по сборке netpolice

[devel] Замечания по сборке netpolice

[Igor Zubkov]

Hi!

Замечания по сборке netpolice:
http://git.altlinux.org/people/letanton/packages/?p=netpolice.git;a=blob;f=netpolice.spec;h=e1bab3615011ced7b102560a655fe8490b3dd945;hb=4fd519f4a4f3e620197f1c22f9ac7bbbf4793d59

1)  Summary: NETPOLICE IS CONTENT FILTERING SYSTEM

Не стоит писать так. Это выглядит так как будто тебе кричат на ухо.

2)  Release: alt2.1

Странный релиз, учитывая что в %changelog одна запись:
  49 %changelog
  50 * Mon Mar 1 2010 Anton Pischulin <letanton@altlinux.ru> 1.02-alt2.1
  51 - Change version to 1.01

3)   13 Conflicts: %name < %version-%release
  14 Conflicts: %name > %version-%release

Это выглядит странно. Т.е. мы у пакета netpolice ставим конфликты на
самого себе только других версий. Зачем?

4)   16 %description
  17 NETPOLICE ID CONTENT FILTERING SYSTEM.

Опять же, не надо всё писать так. К тому же, здесь опечатка. "ID" хотя
подразумевалось "IS".

5)   30 Provides: %name-main = %version-%release, %_docdir/%name/%name

Пакет %name-main провайдит %name-main. RPM это и так сделает. Зачем
ещё провайдить %_docdir/%name/%name?

6)   34 %description -n %name-main
  35 This package is meta package for %name

У описания пакета принято ставить точку в конце.

7)   41 %post
  42 %post -n %name-main
  43 /sbin/chkconfig --level 345 squid on
  44 /sbin/chkconfig --level 345 c-icap on

%post на 41 строчке вообще лишний.

Ну и второе, а добавлять чужие сервисы в загрузку это нормально?

-- 
Igor Zubkov
http://hi.im/ice

Re: [devel] Замечания по сборке netpolice

[Michael Shigorin]

On Tue, Nov 09, 2010 at 01:56:51PM +0200, Igor Zubkov wrote:
> Ну и второе, а добавлять чужие сервисы в загрузку это нормально?

Нет.

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

Re: [devel] Замечания по сборке netpolice

[Anton V. Boyarshinov]

> 5)   30 Provides: %name-main = %version-%release, %_docdir/%name/%name
Requires: %name = %version-%release 
в netpolice-filter тоже выглядит небанально.

И в результате всех этих множественных версионированных requires и
conflicts имеем:
The following packages have unmet dependencies:
  netpolice-main: Depends: netpolice-filter (>= 1.01-alt2.1)
                  Depends: host2cat (>= 1.01-alt2.1)