<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>14718</bug_id>
          
          <creation_ts>2008-03-01 06:15:20 +0300</creation_ts>
          <short_desc>[FR] %name-buildroot test</short_desc>
          <delta_ts>2008-03-03 08:11:20 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>rpm-build</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>REOPENED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Evgenii Terechkov">evg</reporter>
          <assigned_to name="placeholder@altlinux.org">placeholder</assigned_to>
          <cc>arseny</cc>
    
    <cc>glebfm</cc>
    
    <cc>imz</cc>
    
    <cc>ldv</cc>
    
    <cc>php-coder</cc>
    
    <cc>placeholder</cc>
    
    <cc>viy</cc>
    
    <cc>vt</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>64875</commentid>
    <comment_count>0</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2008-03-01 06:15:20 +0300</bug_when>
    <thetext>from devel@:
===============================================================================
P.S.: дарю идею - сделать юниттест, проверяющий все файлы пакета на предмет
текстового вхождения %name-buildroot (имени каталога, где происходила
сборка). Думаю, это было бы сильно полезнее против гораздо более
разломанных пакетов. Или это на sisyphus_check?
===============================================================================

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

Я больше склоняюсь к тому, чтобы по результату этого теста выдавать
предупреждение. Вот пример перед глазами - пакеты с байт-кодом для emacs
ссылаются на исходные *.el-файлы в %name-buildroot если они есть, иначе
прозрачно переключаются на нужные .el-файлы (вижу на примере emacs-jabber).
Насколько я понял, это не чинится.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64886</commentid>
    <comment_count>1</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2008-03-01 15:58:10 +0300</bug_when>
    <thetext>&gt; Временами попадаются такие больные на голову системы сборки, которые пути сборки
&gt; в итоговые файлы записывают. Временами это вылезает очень неприятной
&gt; неработоспособностью пакетов.

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

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

Иногда не чинится.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64893</commentid>
    <comment_count>2</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2008-03-01 17:32:53 +0300</bug_when>
    <thetext>(In reply to comment #1)
&gt; &gt; Насколько я понял, это не чинится
&gt; Иногда не чинится.

В данном частном случае - байт-код емакса. Ну а чаще такое можно и нужно чинить.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64908</commentid>
    <comment_count>3</comment_count>
    <who name="viy">viy</who>
    <bug_when>2008-03-01 21:13:28 +0300</bug_when>
    <thetext>тест соответствующий я написал, 
результаты опубликую в devel@.
но как показал анализ.
не всегда такое вхождение является ошибкой.

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

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64909</commentid>
    <comment_count>4</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2008-03-01 21:15:21 +0300</bug_when>
    <thetext>Пусть теперь на rpm-build повисит.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64911</commentid>
    <comment_count>5</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2008-03-01 21:30:54 +0300</bug_when>
    <thetext>(In reply to comment #3)
&gt; тест соответствующий я написал, 
&gt; результаты опубликую в devel@.
&gt; но как показал анализ.
&gt; не всегда такое вхождение является ошибкой.

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

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

А мне вот интересно, что по этому поводу думает ldv@. И да, скорее это уж на
rpm-build стойло бы повесить. А дублирование с результатами repocop, я думаю,
небезсмысленно - т.к. вывод rpm-build в общем случае не публикуется.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64912</commentid>
    <comment_count>6</comment_count>
    <who name="viy">viy</who>
    <bug_when>2008-03-01 21:42:12 +0300</bug_when>
    <thetext>rpm-build может избить невиновных...</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>