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

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

[Slava Semushin]

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

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

[Vitaly Kuznetsov]

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

Это я предложеный rpmcs тестирую ;)
не думал что он будет в changelog-е что-то менять

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

[Slava Semushin]

2007/7/9, Vitaly Kuznetsov <vitty / altlinux.ru>:
[...]
> Это я предложеный rpmcs тестирую ;)
> не думал что он будет в changelog-е что-то менять

Программы не совершенны. Алгоритмы не могут предусмотреть всех
ситуаций. Тут нужен человек :)

В данном случае даже изменение в %build ломает задумку автора.

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

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

[Денис Смирнов]

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

On Mon, Jul 09, 2007 at 01:20:34PM +0700, Slava Semushin wrote:

>> +rm -f `find %buildroot -type f -name perllocal.pod -o -name .packlist`
>> +find %buildroot -name API.bs -a -size 0 -exec rm -f {} \;
SS> Как-то нелогично. Почему бы и в первом случае не использовать find +
SS> exec? Или даже в обоиз случаях find -delete

Первая команда это не просто ужас, это ужас-ужас.
exec здесь абсолютно не к месту. Если удалять, то действительно find
-delete.

А в других случаях стоит обратить внимание на find -print0 | xargs -0r
<commandname>.

Так как xargs может получать сразу множество имен файлов в качестве
параметров, не будет fork/exec на _каждый_ файл.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
И Бейсик - сын ошибок трудных, и Клиппер Парадоксов друг.

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

[devel] rm `find` (was: [git update] packages/perl-mozldap: heads/master)

[Michael Shigorin]

On Mon, Jul 09, 2007 at 11:02:55AM +0400, Денис Смирнов wrote:
> А в других случаях стоит обратить внимание на find -print0 | xargs -0r
> <commandname>.  Так как xargs может получать сразу множество
> имен файлов в качестве параметров, не будет fork/exec на
> _каждый_ файл.

Не только, см. spp.tex или
http://docs.altlinux.ru/alt/devel/ch01s03.html

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

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

[Dmitry V. Levin]

[-- 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 --]

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

[Slava Semushin]

09.07.07, Dmitry V. Levin<ldv / altlinux.org> написал(а):
[...]
> Это просто сборник ошибок:

Будем(?) публиковать после публикации архивов Сизифа и трудов vsu@ :)

> - 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

Многие их этих советов нигде не документированы (во всяком случае, я
их не видел), не очевидны (как с файлами перла) и являются
ALT-специфичными. Это стОит записать/добавить куда...

2mike@: :)

> > [...]
> > >  # 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
> >
> > Во-первых, вы тут перестарались с заменой макросов. После вашего
> > изменения оба условия стали одинаковыми.
> >
> > Во-вторых, этот скрипт в АЛЬТе (если не ошибаюсь) запускается
> > автоматически сам и этот код абсолютно не нужен.
>
> Оба замечания верны.

Как видно, идея заимствовать/использовать спеки из других
дистрибутивов не всегда хороша. Во всяком случае, без адаптации под
реалии АЛЬТа. (Хотя да, работать это будет...)


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

[devel] [wiki] devel/TypicalPackagingErrors/FindRm

[Michael Shigorin]

On Mon, Jul 09, 2007 at 01:45:20PM +0400, Dmitry V. Levin wrote:
> > > +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
> Это просто сборник ошибок:

Занесено на
http://www.freesource.info/wiki/AltLinux/Sisyphus/devel/TypicalPackagingErrors/FindRm

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

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

[Anton Farygin]

Dmitry V. Levin wrote:
> 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 . -type f|while read line;do rm -f "$line";done

??

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

[Anton Farygin]

Slava Semushin wrote:
> 09.07.07, Dmitry V. Levin<ldv / altlinux.org> написал(а):
> [...]
>> Это просто сборник ошибок:
> 
> Будем(?) публиковать после публикации архивов Сизифа и трудов vsu@ :)
> 
>> - 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
> 
> Многие их этих советов нигде не документированы (во всяком случае, я
> их не видел), не очевидны (как с файлами перла) и являются
> ALT-специфичными. Это стОит записать/добавить куда...
> 
> 2mike@: :)

ну не правда.. по поводу find - есть куча мест где вылезают грабли с 
таким использованием как у вас.

сразу заметно на файлопомойке, если пытаться делать с файлами на ней 
какие-то операции... например chmod всем файлам.

в этом случае вылезает ругань про то, что файл не найден.. видимо 
пробелы мешаются.

Rgds,
Rider

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

[Dmitry V. Levin]

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

On Mon, Jul 09, 2007 at 08:15:58PM +0400, Anton Farygin wrote:
> Slava Semushin wrote:
> > 09.07.07, Dmitry V. Levin<ldv / altlinux.org> написал(а):
> > [...]
> >> Это просто сборник ошибок:
> > 
> > Будем(?) публиковать после публикации архивов Сизифа и трудов vsu@ :)
> > 
> >> - 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
> > 
> > Многие их этих советов нигде не документированы (во всяком случае, я
> > их не видел), не очевидны (как с файлами перла) и являются
> > ALT-специфичными. Это стОит записать/добавить куда...
> > 
> > 2mike@: :)
> 
> ну не правда.. по поводу find - есть куча мест где вылезают грабли с 
> таким использованием как у вас.

У нас?  Кого из нас ты имеешь в виду?


-- 
ldv

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

Re: [devel] find Security Considerations

[Dmitry V. Levin]

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

On Mon, Jul 09, 2007 at 08:13:28PM +0400, Anton Farygin wrote:
> Dmitry V. Levin wrote:
> > 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 . -type f|while read line;do rm -f "$line";done

В этой конструкции много недостатков.  Я не буду их перечислять, просто
рекомендую ознакомиться с соответствующей документацией,
$ info -f find security

Кстати, немного истории "find -delete":
Этот параметр ведёт свою историю с ноября 1997-го года, когда патч к
findutils-4.1 анонсировал в bugtraq'е Douglas Siebert:
http://marc.info/?l=bugtraq&m=87930069226479

Этот патч был адаптирован для Owl в сентябре 2000-го года,
у нас -- а апреле 2002-го года.
GNU findutils поддерживает параметр -delete с ноября 2004 года
(после того как GNU findutils обрёл нового мантейнера):
http://savannah.gnu.org/patch/index.php?3454


-- 
ldv

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

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

[Anton Farygin]

Dmitry V. Levin wrote:
> On Mon, Jul 09, 2007 at 08:15:58PM +0400, Anton Farygin wrote:
>> Slava Semushin wrote:
>>> 09.07.07, Dmitry V. Levin<ldv / altlinux.org> написал(а):
>>> [...]
>>>> Это просто сборник ошибок:
>>> Будем(?) публиковать после публикации архивов Сизифа и трудов vsu@ :)
>>>
>>>> - 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
>>> Многие их этих советов нигде не документированы (во всяком случае, я
>>> их не видел), не очевидны (как с файлами перла) и являются
>>> ALT-специфичными. Это стОит записать/добавить куда...
>>>
>>> 2mike@: :)
>> ну не правда.. по поводу find - есть куча мест где вылезают грабли с 
>> таким использованием как у вас.
> 
> У нас?  Кого из нас ты имеешь в виду?

Славу естественно.

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

[Slava Semushin]

10.07.07, Anton Farygin<rider / altlinux.com> написал(а):
> Dmitry V. Levin wrote:
> > On Mon, Jul 09, 2007 at 08:15:58PM +0400, Anton Farygin wrote:
> >> Slava Semushin wrote:
> >>> 09.07.07, Dmitry V. Levin<ldv / altlinux.org> написал(а):
> >>> [...]
> >>>> Это просто сборник ошибок:
> >>> Будем(?) публиковать после публикации архивов Сизифа и трудов vsu@ :)
> >>>
> >>>> - 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
> >>> Многие их этих советов нигде не документированы (во всяком случае, я
> >>> их не видел), не очевидны (как с файлами перла) и являются
> >>> ALT-специфичными. Это стОит записать/добавить куда...
> >>>
> >>> 2mike@: :)
> >> ну не правда.. по поводу find - есть куча мест где вылезают грабли с
> >> таким использованием как у вас.
> >
> > У нас?  Кого из нас ты имеешь в виду?
>
> Славу естественно.

А причем тут я?! :) Если не ошибаюсь, в моих спеках нигде нет подобных
контрукций (find+rm), просто потому что не было такой задачи.


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