Bug 41456

Summary: Зависит от systemd
Product: Sisyphus Reporter: Антон Мидюков <antohami>
Component: apache2-baseAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: enhancement    
Priority: P5 CC: aen, rider, shaba
Version: unstable   
Hardware: all   
OS: Linux   
See Also: https://bugzilla.altlinux.org/show_bug.cgi?id=41444
https://bugzilla.altlinux.org/show_bug.cgi?id=41454
https://bugzilla.altlinux.org/show_bug.cgi?id=41455
Attachments:
Description Flags
Патч для скрытия зависимости
none
Патч для скрытия зависимости через переменную
none
Патч для скрытия зависимости через переменную (исправлен отступ)
none
Патч для скрытия зависимости через переменную c указанием пути none

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.