Bug 32554 - переопределяет %_localstatedir
Summary: переопределяет %_localstatedir
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: rpm-build-intro (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Vitaly Lipatov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 10382
  Show dependency tree
 
Reported: 2016-10-01 10:21 MSK by Michael Shigorin
Modified: 2017-10-28 13:23 MSK (History)
7 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2016-10-01 10:21:32 MSK 
Вместо /var/lib получается /var -- дискуссия открыта, но диверсии -- не метод.
Comment 1 Vitaly Lipatov 2016-10-01 15:10:31 MSK 
(В ответ на комментарий №0)
> Вместо /var/lib получается /var -- дискуссия открыта, но диверсии -- не метод.
Дискуссия https://bugzilla.altlinux.org/show_bug.cgi?id=10382 открыта 10 лет. 
Если rpm-build-intro используется осознанно, то можно избавиться от хаков в спеке, или же убрать зависимость на rpm-build-intro.

С другой стороны, в mlocate оказалось проще заменить localstatedir на sharedstatedir и не мучаться:
%__subst 's|$(localstatedir)|$(sharedstatedir)|g' Makefile.*
https://packages.altlinux.org/ru/Sisyphus/srpms/mlocate/spec
Comment 2 Ivan Zakharyaschev 2017-02-20 16:38:57 MSK 
(In reply to comment #0)
> Вместо /var/lib получается /var -- дискуссия открыта, но диверсии -- не метод.

Так это неплохо: мейнтейнер осознанно пишет спек, предполагая особые значения. Готовится к переезду (в будущем).
Comment 3 Vitaly Lipatov 2017-10-25 01:34:47 MSK 
> переопределяет %_localstatedir
> вместо /var/lib получается /var
именно так.
Comment 4 Dmitry V. Levin 2017-10-25 07:52:18 MSK 
Я думаю, что если смена значения системного макроса %_localstatedir в пакете с другими (дополнительными) макросами - это WORKSFORME, то конфликт на rpm-build-intro в пакете rpm-build будет адекватной реакцией.
Comment 5 Dmitry V. Levin 2017-10-25 08:02:11 MSK 
Пользователь, который установил пакет ради дополнительных макросов, вправе рассчитывать, что системные макросы не пострадают.

Предлагаю завести отдельный пакет с выразительным именем, например, rpm-macros-[in]compat-fedora, и помещать туда макросы из федоры, конфликтующие с системными, вместо того, чтобы смешивать в одном пакете конфликтующие и дополнительные макросы.
Comment 6 Anton Farygin 2017-10-25 08:24:35 MSK 
если уж у нас не будет исправлен _localstatedir то я тоже предпочитаю прописывать его вручную в каждом спеке, вместо того, что бы устанавливать переопределяющий системные макросы пакет.
Comment 7 Vitaly Lipatov 2017-10-25 13:34:08 MSK 
(In reply to comment #5)
> Пользователь, который установил пакет ради дополнительных макросов, вправе
> рассчитывать, что системные макросы не пострадают.
> 
> Предлагаю завести отдельный пакет с выразительным именем, например,
> rpm-macros-[in]compat-fedora, и помещать туда макросы из федоры, конфликтующие
> с системными, вместо того, чтобы смешивать в одном пакете конфликтующие и
> дополнительные макросы.
Спасибо, вынесу в отдельный пакет.
Comment 8 Sergey Y. Afonin 2017-10-25 16:05:11 MSK 
Раз bug 10382 закрыт так, как закрыт, то не плохо бы было отвязать etersoft-build-utils от пакетов, которые переопределяют localstatedir.
Comment 9 Vitaly Lipatov 2017-10-28 13:21:19 MSK 
(В ответ на комментарий №8)
> Раз bug 10382 закрыт так, как закрыт, то не плохо бы было отвязать
> etersoft-build-utils от пакетов, которые переопределяют localstatedir.
Это было сделано в сборке
2017-10-18 Vitaly Lipatov <lav at altlinux.ru> 2.7.2-alt1
Comment 10 Repository Robot 2017-10-28 13:23:24 MSK 
rpm-build-altlinux-compat-2.1.0-alt1 -> sisyphus:

Sat Oct 28 2017 Vitaly Lipatov <lav@altlinux.ru> 2.1.0-alt1
- move _localstatedir macro to rpm-macros-intro-conflicts package (ALT bug 32554)