Re: [devel] makeinstall, make_install, makeinstall_std...

[Alexey Rusakov <ktirf@altlinux.org>]

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

В Вск, 15/02/2009 в 23:57 +0300, Alexey Rusakov пишет:
> Доброго времени суток.

Да, ещё две ссылки по теме - 

http://www.altlinux.org/Spec#.25makeinstall
http://www.altlinux.org/ОсобенностиСборкиПакетов


> Разбирая старую почту, наткнулся на проблему, к которой где-то раз в год
> кто-нибудь делает очередной подход. Речь о
> 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 --]