Bug 13925

Summary: Название макроcа %__autoreconf предполагает, что этот макрос внутренний
Product: Sisyphus Reporter: Mikhail Gusarov <dottedmag>
Component: rpm-buildAssignee: placeholder <placeholder>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P2 CC: arseny, glebfm, imz, ldv, php-coder, placeholder, vt, vvk
Version: unstable   
Hardware: all   
OS: Linux   

Description Mikhail Gusarov 2008-01-07 19:30:03 MSK
Макрос %__autoreconf начинается с __, но при этом разрешён к использванию в 
спек-файлах. Это нарушает неявное правило, соблюдающееся другими макросами: 
"макросы, начинающиеся на __ - внутренние для RPM, и не должны использоваться в 
спек-файлах..

Предлагается для повышения консистентности именования и уменьшения количества 
вопросов в списках рассылки создать макрос %autoreconf, сохранив %__autoreconf 
для обратной совместимости, но не документировать его (при его использовании 
выплёвывать в лог предупреждение о том, что он устарел, если это возможно в 
RPM).
Comment 1 Mikhail Gusarov 2008-01-07 19:31:23 MSK
Не документировать и выплёвывать предупреждение, конечно же, при использовании 
%__autoreconf.
Comment 2 Slava Semushin 2008-01-07 19:50:03 MSK
А %autoreconf тогда будет походить на внутренние секции RPM из спек-файла (типа
%setup/%build)...
Comment 3 Mikhail Gusarov 2008-01-07 19:53:37 MSK
2slava: %make, %setup, %def_enable, %configure, %make_build, %dir, %config - 
они все походят на внутренние секции RPM из спек-файла.
Comment 4 Dmitry V. Levin 2008-01-08 00:13:18 MSK
(In reply to comment #0)
> Макрос %__autoreconf начинается с __, но при этом разрешён к использванию в 
> спек-файлах. Это нарушает неявное правило, соблюдающееся другими макросами: 
> "макросы, начинающиеся на __ - внутренние для RPM, и не должны использоваться в 
> спек-файлах..
> 
> Предлагается для повышения консистентности именования и уменьшения количества 
> вопросов в списках рассылки создать макрос %autoreconf, сохранив %__autoreconf 
> для обратной совместимости, но не документировать его

логично

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

это возможно, но мало кто предупреждение заметит.
Comment 5 Mikhail Gusarov 2008-01-08 00:43:53 MSK
> это возможно, но мало кто предупреждение заметит.

Вреда особого не принесёт, а польза будет ненулевая - кто-нибудь всё равно 
заметит и исправит свой пакет.
Comment 6 Dmitry V. Levin 2008-01-18 23:06:27 MSK
Fixed in 4.0.4-alt83: макрос переименован, предупреждение добавлено.