Bug 45900 - Убрать зависимость на sudo
Summary: Убрать зависимость на sudo
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: brltty (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 enhancement
Assignee: viy
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-04-19 11:46 MSK by Sergey V Turchin
Modified: 2023-04-27 14:59 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 и вдруг перестал?
Да, не собрался у меня. Мне не нужно было делать лишние действия.