Bug 41456 - Зависит от systemd
Summary: Зависит от systemd
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: apache2-base (show other bugs)
Version: unstable
Hardware: all Linux
: P5 enhancement
Assignee: Nobody's working on this, feel free to take it
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-28 19:17 MSK by Антон Мидюков
Modified: 2021-11-29 14:18 MSK (History)
3 users (show)

See Also:


Attachments
Патч для скрытия зависимости (868 bytes, patch)
2021-11-28 19:17 MSK, Антон Мидюков
no flags Details | Diff
Патч для скрытия зависимости через переменную (892 bytes, patch)
2021-11-29 08:47 MSK, Антон Мидюков
no flags Details | Diff
Патч для скрытия зависимости через переменную (исправлен отступ) (896 bytes, patch)
2021-11-29 08:51 MSK, Антон Мидюков
no flags Details | Diff
Патч для скрытия зависимости через переменную c указанием пути (902 bytes, patch)
2021-11-29 09:00 MSK, Антон Мидюков
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Антон Мидюков 2021-11-28 19:17:43 MSK
Created attachment 9988 [details]
Патч для скрытия зависимости

Пакет ранее зависел от systemd-utils, но теперь от systemd, так как с версии 1:249.7-alt3 systemd-utils провайдится от systemd.
Теперь systemd прилетит на системы с sysvinit, на которых установлен этот пакет.
Поэтому требуется оторвать зависимость на systemd.

Зависимость порождается 90-apache2-base-httpd.filetrigger
Comment 1 Anton Farygin 2021-11-29 08:39:01 MSK
не могу в этом никак помочь.

У меня нет установок на apache c sysvinit и я не планирую поддерживать его в этом пакете. 

С удовольствием рассмотрю pull request.
Comment 2 Антон Мидюков 2021-11-29 08:40:28 MSK
(Ответ для Anton Farygin на комментарий #1)
> не могу в этом никак помочь.
> 
> У меня нет установок на apache c sysvinit и я не планирую поддерживать его в
> этом пакете. 
> 
> С удовольствием рассмотрю pull request.

Я же приложил патч.
Comment 3 Anton Farygin 2021-11-29 08:40:46 MSK
патч не совсем годный. Лучше вынести systemctl в переменную.
Comment 4 Антон Мидюков 2021-11-29 08:47:40 MSK
Created attachment 9989 [details]
Патч для скрытия зависимости через переменную

(Ответ для Anton Farygin на комментарий #3)
> патч не совсем годный. Лучше вынести systemctl в переменную.

Переделал. Нормально?
Comment 5 Антон Мидюков 2021-11-29 08:51:40 MSK
Created attachment 9990 [details]
Патч для скрытия зависимости через переменную (исправлен отступ)
Comment 6 Anton Farygin 2021-11-29 08:55:05 MSK
к systemctl надо указать полный путь.
Comment 7 Антон Мидюков 2021-11-29 09:00:31 MSK
Created attachment 9991 [details]
Патч для скрытия зависимости через переменную c указанием пути
Comment 8 Антон Мидюков 2021-11-29 11:25:34 MSK
(Ответ для Anton Farygin на комментарий #6)
> к systemctl надо указать полный путь.

Сделано.
Последняя версия патча норм?
Мне самому задание собрать?
Comment 9 Anton Farygin 2021-11-29 11:34:03 MSK
systemctl у нас в другом месте лежит
Comment 10 Антон Мидюков 2021-11-29 11:43:32 MSK
(Ответ для Anton Farygin на комментарий #9)
> systemctl у нас в другом месте лежит

Там тоже лежит:

$ ls -l /sbin/systemctl 
lrwxrwxrwx 1 root root 16 ноя 24 01:07 /sbin/systemctl -> ../bin/systemctl
Comment 11 Anton Farygin 2021-11-29 11:44:26 MSK
поменяй на /bin/systemctl и можно отправлять задание в репозиторий sisyphus и, затем, в p10.
Comment 12 Repository Robot 2021-11-29 13:21:45 MSK
apache2-1:2.4.51-alt2 -> sisyphus:

 Mon Nov 29 2021 Anton Midyukov <antohami@altlinux.ru> 1:2.4.51-alt2
 - 90-apache2-base-httpd.filetrigger: do not require systemctl (closes: #41456)
Comment 13 Alexey Shabalin 2021-11-29 14:17:17 MSK
(Ответ для Anton Farygin на комментарий #6)
> к systemctl надо указать полный путь.

Но зачем? не надо указывать полный путь. Есть же $PATH.
Comment 14 Anton Farygin 2021-11-29 14:18:13 MSK
Не надо PATH. Я считаю что полный путь в post-скриптах это правильно.

Мало ли какой там sysctemctl будет в PATH.