Bug 14718

Summary: [FR] %name-buildroot test
Product: Sisyphus Reporter: Evgenii Terechkov <evg>
Component: rpm-buildAssignee: placeholder <placeholder>
Status: REOPENED --- QA Contact: qa-sisyphus
Severity: enhancement    
Priority: P2 CC: arseny, glebfm, imz, ldv, php-coder, placeholder, viy, vt
Version: unstable   
Hardware: all   
OS: Linux   

Description Evgenii Terechkov 2008-03-01 06:15:20 MSK
from devel@:
===============================================================================
P.S.: дарю идею - сделать юниттест, проверяющий все файлы пакета на предмет
текстового вхождения %name-buildroot (имени каталога, где происходила
сборка). Думаю, это было бы сильно полезнее против гораздо более
разломанных пакетов. Или это на sisyphus_check?
===============================================================================

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

Я больше склоняюсь к тому, чтобы по результату этого теста выдавать
предупреждение. Вот пример перед глазами - пакеты с байт-кодом для emacs
ссылаются на исходные *.el-файлы в %name-buildroot если они есть, иначе
прозрачно переключаются на нужные .el-файлы (вижу на примере emacs-jabber).
Насколько я понял, это не чинится.
Comment 1 Dmitry V. Levin 2008-03-01 15:58:10 MSK
> Временами попадаются такие больные на голову системы сборки, которые пути сборки
> в итоговые файлы записывают. Временами это вылезает очень неприятной
> неработоспособностью пакетов.

Иногда встречается.

> Я больше склоняюсь к тому, чтобы по результату этого теста выдавать
> предупреждение. Вот пример перед глазами - пакеты с байт-кодом для emacs
> ссылаются на исходные *.el-файлы в %name-buildroot если они есть, иначе
> прозрачно переключаются на нужные .el-файлы (вижу на примере emacs-jabber).
> Насколько я понял, это не чинится.

Иногда не чинится.
Comment 2 Evgenii Terechkov 2008-03-01 17:32:53 MSK
(In reply to comment #1)
> > Насколько я понял, это не чинится
> Иногда не чинится.

В данном частном случае - байт-код емакса. Ну а чаще такое можно и нужно чинить.
Comment 3 viy 2008-03-01 21:13:28 MSK
тест соответствующий я написал, 
результаты опубликую в devel@.
но как показал анализ.
не всегда такое вхождение является ошибкой.

поэтому предлагаю баг закрыть. это не для sisyphus_check.

Comment 4 Dmitry V. Levin 2008-03-01 21:15:21 MSK
Пусть теперь на rpm-build повисит.
Comment 5 Evgenii Terechkov 2008-03-01 21:30:54 MSK
(In reply to comment #3)
> тест соответствующий я написал, 
> результаты опубликую в devel@.
> но как показал анализ.
> не всегда такое вхождение является ошибкой.

Да я и сам об этом же пишу - лучше warning показывать с указанием, чего и где.

> поэтому предлагаю баг закрыть. это не для sisyphus_check.

А мне вот интересно, что по этому поводу думает ldv@. И да, скорее это уж на
rpm-build стойло бы повесить. А дублирование с результатами repocop, я думаю,
небезсмысленно - т.к. вывод rpm-build в общем случае не публикуется.
Comment 6 viy 2008-03-01 21:42:12 MSK
rpm-build может избить невиновных...