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

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

    <bug>
          <bug_id>13925</bug_id>
          
          <creation_ts>2008-01-07 19:30:03 +0300</creation_ts>
          <short_desc>Название макроcа %__autoreconf предполагает, что этот макрос внутренний</short_desc>
          <delta_ts>2008-01-31 16:49:17 +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>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Mikhail Gusarov">dottedmag</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>vt</cc>
    
    <cc>vvk</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>60543</commentid>
    <comment_count>0</comment_count>
    <who name="Mikhail Gusarov">dottedmag</who>
    <bug_when>2008-01-07 19:30:03 +0300</bug_when>
    <thetext>Макрос %__autoreconf начинается с __, но при этом разрешён к использванию в 
спек-файлах. Это нарушает неявное правило, соблюдающееся другими макросами: 
&quot;макросы, начинающиеся на __ - внутренние для RPM, и не должны использоваться в 
спек-файлах..

Предлагается для повышения консистентности именования и уменьшения количества 
вопросов в списках рассылки создать макрос %autoreconf, сохранив %__autoreconf 
для обратной совместимости, но не документировать его (при его использовании 
выплёвывать в лог предупреждение о том, что он устарел, если это возможно в 
RPM).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60544</commentid>
    <comment_count>1</comment_count>
    <who name="Mikhail Gusarov">dottedmag</who>
    <bug_when>2008-01-07 19:31:23 +0300</bug_when>
    <thetext>Не документировать и выплёвывать предупреждение, конечно же, при использовании 
%__autoreconf.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60546</commentid>
    <comment_count>2</comment_count>
    <who name="Slava Semushin">php-coder</who>
    <bug_when>2008-01-07 19:50:03 +0300</bug_when>
    <thetext>А %autoreconf тогда будет походить на внутренние секции RPM из спек-файла (типа
%setup/%build)...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60548</commentid>
    <comment_count>3</comment_count>
    <who name="Mikhail Gusarov">dottedmag</who>
    <bug_when>2008-01-07 19:53:37 +0300</bug_when>
    <thetext>2slava: %make, %setup, %def_enable, %configure, %make_build, %dir, %config - 
они все походят на внутренние секции RPM из спек-файла.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60557</commentid>
    <comment_count>4</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2008-01-08 00:13:18 +0300</bug_when>
    <thetext>(In reply to comment #0)
&gt; Макрос %__autoreconf начинается с __, но при этом разрешён к использванию в 
&gt; спек-файлах. Это нарушает неявное правило, соблюдающееся другими макросами: 
&gt; &quot;макросы, начинающиеся на __ - внутренние для RPM, и не должны использоваться в 
&gt; спек-файлах..
&gt; 
&gt; Предлагается для повышения консистентности именования и уменьшения количества 
&gt; вопросов в списках рассылки создать макрос %autoreconf, сохранив %__autoreconf 
&gt; для обратной совместимости, но не документировать его

логично

&gt; (при его использовании 
&gt; выплёвывать в лог предупреждение о том, что он устарел, если это возможно в 
&gt; RPM).

это возможно, но мало кто предупреждение заметит.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60558</commentid>
    <comment_count>5</comment_count>
    <who name="Mikhail Gusarov">dottedmag</who>
    <bug_when>2008-01-08 00:43:53 +0300</bug_when>
    <thetext>&gt; это возможно, но мало кто предупреждение заметит.

Вреда особого не принесёт, а польза будет ненулевая - кто-нибудь всё равно 
заметит и исправит свой пакет.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>61388</commentid>
    <comment_count>6</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2008-01-18 23:06:27 +0300</bug_when>
    <thetext>Fixed in 4.0.4-alt83: макрос переименован, предупреждение добавлено.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>