Bug 45900

Summary: Убрать зависимость на sudo
Product: Sisyphus Reporter: Sergey V Turchin <zerg>
Component: brlttyAssignee: Artem Semenov <savoptik>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: enhancement    
Priority: P5 CC: savoptik, viy
Version: unstable   
Hardware: x86_64   
OS: Linux   

Description Sergey V Turchin 2023-04-19 11:46:24 MSK 
По умолчанию он не использует sudo.
В нём нет нигде инструкций использовать параметр S.
Предлагаю оторвать зависимость на sudo.
Comment 1 Yuri N. Sedunov 2023-04-19 12:34:58 MSK 
Я что-то пропустил, мы теперь боремся с sudo?
Comment 2 Sergey V Turchin 2023-04-19 13:29:55 MSK 
(Ответ для Yuri N. Sedunov на комментарий #1)
> Я что-то пропустил, мы теперь боремся с sudo?
Почему бы и нет?
Comment 3 Sergey V Turchin 2023-04-19 13:31:23 MSK 
На самом деле у меня в дистрибутиве нет зависимостей на sudo и не хочу добавлять, а возникла необходимость добавить orca и он притащил за собой.
Comment 4 Sergey V Turchin 2023-04-19 13:39:16 MSK 
(Ответ для Yuri N. Sedunov на комментарий #1)
> Я что-то пропустил, мы теперь боремся с sudo?
А так, мне лично не нравится именно _навязывание_ sudo людьми с убунтой головного мозга.
Comment 5 Sergey V Turchin 2023-04-19 14:25:45 MSK 
Ааа, acl всем есть. Я сам тогда.
Comment 6 Sergey V Turchin 2023-04-19 14:28:56 MSK 
Ой, там srpm. Сконвертирую в git?
Comment 7 Yuri N. Sedunov 2023-04-19 14:31:42 MSK 
Эка, тебя плющит :)
Ты хоть знаешь что делать? Без конвертирования в git никак?
Comment 8 Sergey V Turchin 2023-04-19 14:37:36 MSK 
(Ответ для Yuri N. Sedunov на комментарий #7)
> Без конвертирования в git никак?
Как только повяился git, не пробовал ни разу srpm собирать по этой новой схеме. Не умею и начинать не особо хочется.
Comment 9 Yuri N. Sedunov 2023-04-19 14:45:21 MSK 
Если знаешь, что делать -- можешь прислать мне патч, изготовленный в git'е с внятным коммит-месседжем, и я его приложу в src.rpm.
Подсказка:

У brltty нет явной зависимости на sudo, она появляется из этих двух скриптов, в которых sudo нужно замаскировать под переменную.

Справишься?

$ grep -i sudo -r `rpmql brltty`
/usr/bin/brltty-mkuser:   "${useSudo}" && set -- sudo -- "${@}"
/usr/bin/brltty-mkuser:addProgramOption S flag useSudo "use sudo to execute the commands as root"
/usr/bin/brltty-mkuser:   if "${useSudo}"
/usr/bin/brltty-mkuser:      sudo -v
/usr/bin/brltty-setcaps:   "${useSudo}" && set -- sudo -- "${@}"
/usr/bin/brltty-setcaps:addProgramOption S flag useSudo "use sudo to execute the commands as root"
/usr/bin/brltty-setcaps:   if "${useSudo}"
/usr/bin/brltty-setcaps:      sudo -v
Comment 10 Sergey V Turchin 2023-04-19 15:09:43 MSK 
(Ответ для Yuri N. Sedunov на комментарий #9)
> У brltty нет явной зависимости на sudo, она появляется из этих двух
> скриптов,
Я так и написал изначально.

> в которых sudo нужно замаскировать под переменную.
Достаточно в spec
%filter_from_requires /^sudo$/d

Или надо с маскировкой?
Comment 11 Sergey V Turchin 2023-04-27 12:31:54 MSK 
Так, что скажешь?
Comment 12 Repository Robot 2023-04-27 14:40:05 MSK 
brltty-6.5-alt1.1 -> sisyphus:

 Thu Apr 27 2023 Sergey V Turchin <zerg@altlinux> 6.5-alt1.1
 - NMU: don't require sudo (closes: 45900)
 - NMU: disable LTO to fix compile
Comment 13 Yuri N. Sedunov 2023-04-27 14:49:58 MSK 
(Ответ для Repository Robot на комментарий #12)
> brltty-6.5-alt1.1 -> sisyphus:
> 
>  Thu Apr 27 2023 Sergey V Turchin <zerg@altlinux> 6.5-alt1.1
>  - NMU: don't require sudo (closes: 45900)
>  - NMU: disable LTO to fix compile

Собирался пакет с lto и вдруг перестал?
Comment 14 Sergey V Turchin 2023-04-27 14:59:09 MSK 
(Ответ для Yuri N. Sedunov на комментарий #13)
> Собирался пакет с lto и вдруг перестал?
Да, не собрался у меня. Мне не нужно было делать лишние действия.