Re: [devel] [git update] packages/perl-mozldap: heads/master

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

2007/7/9, Vitaly Kuznetsov <vitty / altlinux.org>:
[...]
> +rm -f `find %buildroot -type f -name perllocal.pod -o -name .packlist`
> +find %buildroot -name API.bs -a -size 0 -exec rm -f {} \;

Как-то нелогично. Почему бы и в первом случае не использовать find +
exec? Или даже в обоиз случаях find -delete

[...]
>  # find and run the correct version of brp-compress
> -if [ -x /usr/lib/rpm/brp-compress ] ; then
> -    /usr/lib/rpm/brp-compress
> +if [ -x %_libdir/rpm/brp-compress ] ; then
> +    %_libdir/rpm/brp-compress
>  elif [ -x %_libdir/rpm/brp-compress ] ; then
>      %_libdir/rpm/brp-compress
>  fi

Во-первых, вы тут перестарались с заменой макросов. После вашего
изменения оба условия стали одинаковыми.

Во-вторых, этот скрипт в АЛЬТе (если не ошибаюсь) запускается
автоматически сам и этот код абсолютно не нужен.

[...]
>  * Tue Oct 17 2006 Rich Megginson <richm@stanfordalumni.org> - 1.5-6
> -- look for brp-compress first in /usr/lib then _libdir
> +- look for brp-compress first in %_libdir then _libdir
[...]

Я бы не стал править записи в %changelog'е, которые делал не я сам
(максимум удалить пробелы на концах строк, но не более).


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