[devel] makeinstall, make_install, makeinstall_std...

[Alexey Rusakov <ktirf@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 2951 bytes --]

Доброго времени суток.

Разбирая старую почту, наткнулся на проблему, к которой где-то раз в год
кто-нибудь делает очередной подход. Речь о
https://bugzilla.altlinux.org/show_bug.cgi?id=13937 - предлагается
добавить порядка в макросы, используемые в секции %install спекфайлов.
Что имеем на сегодня:
* макрос %make_install - это просто указание конкретного вызова make,
используемого в конструкциях вида make install.
* макрос %_makeinstall_target - это цель для make, обычно используемая
для установки, то есть install.
(это всё присказка была, дальше сказка начинается)
* макрос %makeinstall - вызывает make install (если быть более точным, %
__make INSTALL="%install -p" %_makeinstall_target) с переопределёнными
prefix, exec_prefix, bindir, datadir и проч. Знающие люди говорят, что
это довольно кривое наследие Mandrake, обладающее нежелательными
побочными эффектами, сам я на эти побочные эффекты нарывался не помню
сколько раз. Для некоторых исходников, тем не менее, работает, посему в
спекфайлах встречается. Есть робкое предположение, что подавляющее
большинство использований этого макроса можно заменить следующим.
* макрос %makeinstall_std - это в общем-то каноническая конструкция
%make_install DESTDIR=%buildroot install
Встречается (как в виде макроса, так и в развёрнутом виде) очень часто.
Использование в виде макроса ограничивается тем фактом, что в rpm-build
%makeinstall_std не входит, а входит одновременно в rpm-build-compat и
rpm-build-perl, в которых, по счастливому стечению обстоятельств,
определён одинаково.

Хочется сдвинуть дело с мёртвой точки. %makeinstall, насколько я понимаю
- очевидный пережиток, и его пора делать deprecated. %makeinstall_std,
мытьём или катаньем - используется, и его пора вносить в число
"официальных", определённым как
%make_install DESTDIR=%buildroot %{?_makeinstall_target:
%{_makeinstall_target}}

Последствия предлагаемого:
1. (-) Предупреждение - не ошибка, но так или иначе исправление спеков -
это неудобство. Возможно, это предмет для деятельности repocop, но
проблема в том, что
2. (-) вообще говоря, нельзя уверенно сказать, что простая замена %
makeinstall на %makeinstall_std не сломает сборку. Более того, скорее
всего будут прецеденты, когда сломает.
2. (+) Семантика макросов (пока) не меняется, соответственно не меняется
и поведение.
3. (+) Новичкам можно будет не раздумывая говорить "В %install скорее
всего прокатит %makeinstall_std" и не ждать их возвращения с вопросами
из-за того, что они забыли добавить в билдреки rpm-build-compat
4. (+) Вместо двух определений одного и того же макроса у нас будет
одно, и большое количество спекфайлов уменьшится на несколько десятков
байт благодаря выбрасыванию соответствующих BuildRequires и замене
развёрнутого макроса свёрнутым.

Спасибо, что дочитали до этого места. Отдельное спасибо, если
выскажетесь.

-- 
  Alexey "Ktirf" Rusakov
  GNOME Project
  ALT Linux Team

[-- Attachment #2: Эта часть сообщения подписана цифровой подписью --]
[-- Type: application/pgp-signature, Size: 197 bytes --]