Re: [devel] I: mate-desktop: new dep

[Ivan Zakharyaschev <imz@altlinux.org>]

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


On Sun, 10 Apr 2016, Alexey Tourbin wrote:

> 2016-04-10 0:57 GMT+03:00 Ivan Zakharyaschev <imz@altlinux.org>:
>> On Sat, 9 Apr 2016, Alexey Tourbin wrote:
>>> 2016-04-08 17:09 GMT+03:00 Ivan Zakharyaschev <imz@altlinux.org>:
>>>> У mate-desktop-synaptics как бы автообнаружилась новая зависимость из-за
>>>> символической ссылки на /usr/share/X11/xorg.conf.d/50-synaptics.conf :
>>> А почему она раньше не обнаруживалась?
>>
>> Там проверка на пустоту списка файловв find-requires применялась не совсем к
>> тому/не совсем там, как задумано.
>>
>> Так что если в пакете были только символические ссылки (абсолютные), то
>> автозависимости в нём дальше не искались.
>
> Пожалуйста подробнее, с номерами строк.

Я думал, Вы уже знаете, какие строчки были затронуты в обсуждаемых 
измененеях.

Эта проверка на строке 75 -- 
http://git.altlinux.org/gears/r/rpm.git?p=rpm.git;a=blob;f=autodeps/linux.req.in;h=0925c6848e81fdd2ee83af49fbe44f4a6d49b6c3;hb=484d6f523f53389bba6e9a43ca97b7f758030b64#l75 
.

Но не все файлы из списка файлов пакета, которые нужно обработать для 
поиска зависимостей записаны в этот момент в "$workdir"/files. Часть из 
них уже была сразу записана в "$workdir"/files+types в строке 70 , а 
остальные будут добавлены туда только в строке 81.

Исправление -- 
http://git.altlinux.org/gears/r/rpm.git?p=rpm.git;a=commitdiff;h=95ffed16a656532cc87e8ebf25fffb235f994e4b 
.

>> Я бы не драматизировал этот случай в том эмоциональном духе, как тут Миша
>> высказался. (Миша заметил rpm среди обновлений, и я прокомментировал
>> изменения. Вот эта бага, не замеченная долгое время, ещё с коммита at@ 2011
>> или 2009 -- сейчас точно не помню, просто потому что редко случалось -- вот
>> они эти три пакета. А обнаружилось вместе с другим изменением, потому что
>> этот code path стал задействован чаще. Другое изменение: если символическая
>> ссылка ведёт гарантированно -- внутри одного пакета -- на файл с настоящим
>> содержимым, то обратывается она генератором зависимостей как файл того типа.
>> Потому что при работе пакета она и будет всегда служить как файл того типа.)

-- 
Best regards,
Ivan