Bug 49904

Summary: Пакет управления яркостью мониторов brightness-controll
Product: New/proposed packages Reporter: Maxim <max_inc>
Component: Обычный репозиторийAssignee: Andrey Cherepanov <cas>
Status: NEW --- QA Contact: Andrey Cherepanov <cas>
Severity: minor    
Priority: P5 CC: fokanovama, max_inc, rider, viy
Version: не указана   
Hardware: x86_64   
OS: Linux   
Attachments:
Description Flags
Ссылка на репозиторий программы none

Description Maxim 2024-04-04 21:44:31 MSK
Created attachment 15793 [details]
Ссылка на репозиторий программы

Прошу рассмотреть возможность добавления в репозиторий ПО следующего пакета:
https://github.com/LordAmit/Brightness

Данная программ позволяет управлять яркостью мониторов стационарного ПК, что позволяет создавать комфортные условия для работы в условиях недостаточной освещённости.

Установка данного ПО с помощью pip3 показала её работоспособность, при этом есть нюансы:
- созданные ярлыки скрываются
- из автозапуска удаляется
Comment 1 Mari 2024-09-10 15:39:07 MSK
Прошу посмотреть мой вариант подготовки пакета:
http://git.altlinux.org/people/mari/packages/Brightness.git

Обсуждение в заявке на Join #45841.
Comment 2 Anton Farygin 2024-09-10 15:42:10 MSK
  1 tar: .
   2 spec: .gear/brightness.spec

первая ошибка - не надо класть в тарболл дерево
Лучше упаковать апстримный тэг а наши изменения положить diff'ом
Comment 3 Anton Farygin 2024-09-10 15:51:34 MSK
brightness_controller_linux - очень странный name. Я бы назвал его так, как называет апстрим.
Как вы предполагаете нам вычислять что этот пакет соответствует апстримному имени ?
Comment 4 Mari 2024-09-10 16:17:37 MSK
(Ответ для Anton Farygin на комментарий #3)
> brightness_controller_linux - очень странный name. Я бы назвал его так, как
> называет апстрим.
> Как вы предполагаете нам вычислять что этот пакет соответствует апстримному
> имени ?

Изначально имя пакета взято из инструкций pyproject.toml проекта.
При сборке на этапе %doc вываливается ошибка с некорректным путём к каталогам, которые должны совпадать с name в spec-файле.

Нет такого файла или каталога: /tmp/.private/alt/brightness-controller-linux-buildroot/usr/lib/python3/site-packages/brightness-controller-linux
<...>/brightness-controller-linux-2.4.dist-info

По этому пути формируются каталоги, в названии которых нижнее подчеркивание. Поэтому пакет переименован. Есть другое решение?
Comment 5 Mari 2024-09-10 16:21:04 MSK
(Ответ для Anton Farygin на комментарий #3)
> brightness_controller_linux - очень странный name. Я бы назвал его так, как
> называет апстрим.
> Как вы предполагаете нам вычислять что этот пакет соответствует апстримному
> имени ?

Изначально имя пакета взято из инструкций pyproject.toml проекта.
При сборке на этапе %doc вываливается ошибка с некорректным путём к каталогам, которые должны совпадать с name в spec-файле.

Нет такого файла или каталога: /tmp/.private/alt/brightness-controller-linux-buildroot/usr/lib/python3/site-packages/brightness-controller-linux
<...>/brightness-controller-linux-2.4.dist-info

По этому пути формируются каталоги, в названии которых нижнее подчеркивание. Поэтому пакет переименован. Есть другое решение?
Comment 6 Anton Farygin 2024-09-10 16:24:36 MSK
В install секции вообще какая-то ерунда, простите. Надо бы поправить.
Comment 7 Anton Farygin 2024-09-10 16:26:04 MSK
(Ответ для Mari на комментарий #5)
> (Ответ для Anton Farygin на комментарий #3)
> > brightness_controller_linux - очень странный name. Я бы назвал его так, как
> > называет апстрим.
> > Как вы предполагаете нам вычислять что этот пакет соответствует апстримному
> > имени ?
> 
> Изначально имя пакета взято из инструкций pyproject.toml проекта.
> При сборке на этапе %doc вываливается ошибка с некорректным путём к
> каталогам, которые должны совпадать с name в spec-файле.
> 
> Нет такого файла или каталога:
> /tmp/.private/alt/brightness-controller-linux-buildroot/usr/lib/python3/site-
> packages/brightness-controller-linux
> <...>/brightness-controller-linux-2.4.dist-info
> 
> По этому пути формируются каталоги, в названии которых нижнее подчеркивание.
> Поэтому пакет переименован. Есть другое решение?

У вас же на JOIN есть ментор, он предназначен для того, что бы отвечать на такие вопросы. Обсудите это с ним, пожалуйста.

И ещё у вас changelog пакета не соответствует принятым рекомендациям, описанным на www.altlinux.org - рекомендую поправить.
Comment 8 Anton Farygin 2024-09-10 16:39:49 MSK
Ну и что касается самой программы - лучше её функции интегрировать в оконный менеджер.
Она через ddcutil отправляет настройки на экран.
Comment 9 Anton Farygin 2024-09-10 16:42:08 MSK
И в целом, из моих трёх мониторов ddcutil поддерживает ноль ;(
Comment 10 Mari 2024-09-10 17:20:39 MSK
Спасибо, обращусь :) У меня на мониторах работает, в меню отображается с корректным названием.
Comment 11 Mari 2024-09-11 19:26:16 MSK
Начала заново. Обновила репо
http://git.altlinux.org/people/mari/packages/Brightness.git

Название пакета прописано в pyproject.toml. Можно ли его сократить в данном случае до brightness?
Comment 12 Anton Farygin 2024-09-12 08:34:34 MSK
к specfile для начала просмотра нужно применить утилиту cleanup_spec

+cd brightness-controller-linux/ - вот этого выполняться не должно.

Было бы отлично, если бы ментор сделал качественно review прежде чем выкладывать это на публику.

Выбор группы rpm тоже надо бы пересмотреть.
Comment 13 Mari 2024-09-12 12:12:06 MSK
Подождём ментора. Переход обусловлен структурой источника.