[devel] makeinstall, make_install, makeinstall_std...
Alexey Rusakov
=?iso-8859-1?q?ktirf_=CE=C1_altlinux=2Eorg?=
Пн Фев 16 00:01:59 MSK 2009
В Вск, 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
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип : application/pgp-signature
Размер : 197 байтов
Описание: Эта часть сообщения подписана цифровой подписью
Url : <http://lists.altlinux.org/pipermail/devel/attachments/20090216/2e2e2b86/attachment.bin>
Подробная информация о списке рассылки Devel