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

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

[Slava Semushin]

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

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

[Igor Zubkov]

В сообщении от Tuesday 03 July 2007 11:39:57 Slava Semushin написал(а):
> > +make install DESTDIR=%{buildroot} INSTALL="install -p"
>
> %make_install DESTDIR=%buildroot install

Кстати, %make_install как раз и раскрывает как make INSTALL="install -p".

> > +%{_var}/log/varnish
> > +%{_mandir}/man1/*.1*
> > +%{_mandir}/man7/*.7*
>
> Можно заменить на макросы %_logdir, %_man1dir и %_man7dir.

Одно общее замечание, {} лучше вообще убрать. Это можно сделать через 
cleanup_spec из пакета rpm-utils.

> > +%post libs -p /sbin/ldconfig
> > +
> > +%postun libs -p /sbin/ldconfig
>
> [...]
>
> Наверняка, есть скрипты и макросы для автоматизации этого процесса.

%post libs -p %post_ldconfig
%postun libs -p %postun_ldconfig

Как пример для этого, можно привести пакет plib. Кстати, /sbin/ldconfig на 
прямую лучше не дёргать. Это написано ещё в доке к нашему rpm.

-- 
icesik