Bug 13925

Summary:	Название макроcа %__autoreconf предполагает, что этот макрос внутренний
Product:	Sisyphus	Reporter:	Mikhail Gusarov <dottedmag>
Component:	rpm-build	Assignee:	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: макрос переименован, предупреждение добавлено.