Bug 13925 - Название макроcа %__autoreconf предполагает, что этот макрос внутренний
Summary: Название макроcа %__autoreconf предполагает, что этот макрос внутренний
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: rpm-build (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: placeholder@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-07 19:30 MSK by Mikhail Gusarov
Modified: 2008-01-31 16:49 MSK (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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: макрос переименован, предупреждение добавлено.