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

["Dmitry V. Levin" <ldv@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 1953 bytes --]

On Mon, Jul 09, 2007 at 01:20:34PM +0700, Slava Semushin wrote:
> 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

Это просто сборник ошибок:
- rm -f `find ...` это небезопасно в принципе, забудьте эту конструкцию
  навсегда.
- find ... -exec rm -f {} \; это небезопасно в принципе, забудьте эту
  конструкцию навсегда.
- для удаления файлов используйте find ... -delete
- все файлы по имени .packlist внутри %buildroot/usr/lib/perl5/ удаляются
  автоматически в /usr/lib/rpm/brp-cleanup
- все файлы нулевого размера по имени *.bs внутри
  %buildroot/usr/lib/perl5/ удаляются автоматически в
  /usr/lib/rpm/brp-cleanup

> [...]
> >  # 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'е, которые делал не я сам
> (максимум удалить пробелы на концах строк, но не более).

+ заквотировать макросы, которые не должны раскрыться или которых не
существует.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]