Re: [devel] [git update] packages/varnish: heads/master

["Slava Semushin" <slava.semushin@gmail.com>]

2007/7/3, Sergey Ivanov <seriv / altlinux.org>:
[...]
> diff --git a/varnish.spec b/varnish.spec
> new file mode 100644
> index 0000000..23b16fd
> --- /dev/null
> +++ b/varnish.spec
> @@ -0,0 +1,218 @@
> +Summary: Varnish is a high-performance HTTP accelerator
> +Name: varnish
> +Version: 1.0.4.3
> +Release: alt2
> +License: BSD-like
> +Group: System/Servers
> +Packager: Sergey Ivanov <seriv@altlinux.ru>
> +Group: System/Servers

Тег Group указан два раза. Одного было бы достаточно.

[...]
> +%prep
> +%setup -q

s/%setup -q/%setup/

> +
> +%build
> +%__aclocal
> +%__libtoolize --copy --force
> +%__autoheader
> +%__automake --add-missing --copy --foreign
> +%__autoconf

Возможно, вместо этих пяти строк будет достаточно вызова макроса %__autoreconf

> +# Remove "--disable static" if you want to build static libraries
> +# (ie for the devel package)
> +%configure --disable-static

У нас обычно это делается через %def_{enable,disable} Гляньте, к
примеру, как это сделано у меня в libtap.spec

[...]
> +make install DESTDIR=%{buildroot} INSTALL="install -p"

%make_install DESTDIR=%buildroot install

[...]
> +%clean
> +rm -rf %{buildroot}

Секция %clean не нужна.

> +%files
> +%defattr(-,root,root,-)

Если не ошибаюсь, то этот %defattr тоже не нужен, так как оно так по умолчанию.

[...]
> +%{_var}/log/varnish
> +%{_mandir}/man1/*.1*
> +%{_mandir}/man7/*.7*

Можно заменить на макросы %_logdir, %_man1dir и %_man7dir.

[...]
> +%files libs
> +%defattr(-,root,root,-)

Опять же я бы убрал этот %defattr

[...]
> +%post
> +/sbin/chkconfig --add varnish
> +/sbin/chkconfig --add varnishlog
> +
> +%preun
> +if [ $1 -lt 1 ]; then
> +  /sbin/service varnish stop > /dev/null 2>/dev/null
> +  /sbin/service varnishlog stop > /dev/null 2>/dev/null
> +  /sbin/chkconfig --del varnish
> +  /sbin/chkconfig --del varnishlog
> +fi
> +
> +%postun
> +if [ $1 -ge 1 ]; then
> +  /sbin/service varnish condrestart > /dev/null 2>/dev/null
> +  /sbin/service varnishlog condrestart > /dev/null 2>/dev/null
> +fi
> +
> +%post libs -p /sbin/ldconfig
> +
> +%postun libs -p /sbin/ldconfig
[...]

Наверняка, есть скрипты и макросы для автоматизации этого процесса. В
частности для остановки/запуска демонов есть %post_service, кажется.
Советую посмотреть в чужие спеки, как это сделано у других.


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru