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

[Igor Zubkov <igor.zubkov@gmail.com>]

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