Bug 40164

Created attachment 9403 [details]
SSH

(Ответ для Koi на комментарий #0)
> Ментор: antohami@altlinux.org

Подтверждаю.

Ура! :)

(Ответ для Koi на комментарий #1)
> Создано вложение 9403 [details] [подробности]
> SSH
Ok.

(Ответ для Koi на комментарий #0)
> Создано вложение 9402 [details] [подробности]
> GPG
В этом файле находится целых два gpg ключа, одного будет вполне достаточно.

Created attachment 9465 [details]
GPG1

Исправленный ключ

Ни хочу никого отвлекать или лишний раз навязываться, но обо мне случайно не забыли? Подтверждение почты отправлено.

Когда уже кандидату предоставят доступ к git.alt?

Кандидат к git.alt готов. С git умеет работать.

ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.

Один пакет для сборки готов. Прошу предоставить кандидату доступ к сборочнице.

ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.

T/J/S -> 3.4.

Кандидат успешно собрал 6 новых пакетов:
pdfmixtool
radeon-profile
radeon-profile-daemon
tesseract-langpack
crow-translate
qt5ct

Кандидат также успешно обновил до новых мажорных версий пакеты libleptonica и tesseract.

Список собранных заданий:
https://git.altlinux.org/tasks/archive/done/_284/291122/
https://git.altlinux.org/tasks/archive/done/_284/291042/
https://git.altlinux.org/tasks/archive/done/_285/291854/
https://git.altlinux.org/tasks/archive/done/_285/292238/
https://git.altlinux.org/tasks/archive/done/_286/293053/
https://git.altlinux.org/tasks/archive/done/_287/293900/
https://git.altlinux.org/tasks/293954/

Считаю, что кандидат готов к самостоятельной сборке пакетов.

Призван ещё один человек (vseleznv@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.

Общие замечания по собранным пакетам.

По pdfmixtool

Поле Packager избыточно, в общем случае его не стоит использовать.

> License: GPLv3

Неоднозначное обозначение лицензии. Может быть как GPLv3 Only, там и GPLv3 or later (случай данного пакета). Рекомендуется использовать недвусмысленные идентификаторы лицензий (см. [1]).

> Source: %name-%version.tar
> # Source-url: https://gitlab.com/scarpetta/pdfmixtool/-/archive/v%version/pdfmixtool-v%version.tar.gz

Обычно принято комментарии писать перед комментируемой сущностью. Обратный порядок сбивает с толку. Это относится ко всем сборкам.

В проекте используется схема сопровождения исходников, в которых тарбол импортируется репозиторий. При линейной схеме и отсутсвии взятии diff'ов избыточно писать в gear-rules

    tar: v@version@:pdfmixtool

достаточно

    tar: pdfmixtool

И в таком случае сопровождать gear-tags не нужно. При сборках следующих пакетов это было учтено.

По qt5ct:

В changelog'е при оформлении подпунктов принято в конце соответствующего пункта верхнего уровня ставить двоеточие.

По radeon-profile:

Аналогичное замечание про лицензии и комментарии.

Вы уверены, что патчи libleptonica-1.69-alt-debuginfo.patch и libleptonica-alt-makefile.patch больше не нужны? Я внимательно не вглядывался, но я сходу не обнаружил, чтобы в новой версии появились необходимые изменения.

По tesseract:

Изменена схема упаковки исходников, раньше собирались из апстримного гита, кандидат решил, что лучше импортировать из исходников. В commit message импорта написано "source code export v4.1.3", хотя это не экспорт, а импорт :). Лучше для импорта исходников использовать утилиту gear-import(1).

Мне видится неудачной идее зависимость на langpack'и у пакета.

По radeon-profile-daemon:

%post
SYSTEMCTL=systemctl
%post_service radeon-profile-daemon
if [ $1 = 1 ] && sd_booted && "$SYSTEMCTL" --version >/dev/null 2>&1; then
    "$SYSTEMCTL" enable radeon-profile-daemon.service >/dev/null 2>&1 || :
fi

Политика для сервисов по-умолчанию (за небольшим исключение) быть выключенными. Это нужно исправить.

В целом, серьёзных замечаний к кандидату нет. Рекомендую учесть замечания и ответить на вопросы.

Ссылки

[1] https://spdx.org/licenses/

(Ответ для Vladimir D. Seleznev на комментарий #14)
> Вы уверены, что патчи libleptonica-1.69-alt-debuginfo.patch и
> libleptonica-alt-makefile.patch больше не нужны? Я внимательно не
> вглядывался, но я сходу не обнаружил, чтобы в новой версии появились
> необходимые изменения.
Тщательно проверил, патчи действительно еще нужны. Изменю их под текущюю версию.
> По tesseract:
> Мне видится неудачной идее зависимость на langpack'и у пакета.
Tesseract использует для работы языковые модели. Может тогда добавить в сборку модели en\ru?
> По radeon-profile-daemon:
> 
> %post
> SYSTEMCTL=systemctl
> %post_service radeon-profile-daemon
> if [ $1 = 1 ] && sd_booted && "$SYSTEMCTL" --version >/dev/null 2>&1; then
>     "$SYSTEMCTL" enable radeon-profile-daemon.service >/dev/null 2>&1 || :
> fi
> 
> Политика для сервисов по-умолчанию (за небольшим исключение) быть
> выключенными. Это нужно исправить.
Понял, исправлю.

> radeon-profile-daemon

Эта программа входит в состав монитора radeon-profile, если служба отключена, монитор работать не будет. Получается, что программа действительно не будет установлена в работоспособном виде. Не все пользователи сразу догадаются, что сервис нужно включать вручную. Можно ли оставить сервис как есть, или такие случаи не могут быть исключениями? 

Где посмотреть политику сервисов и исключений? Вики нашел Services Policy, но там речь о сетевых.

(Ответ для Vladimir D. Seleznev на комментарий #14)
Все исправил, но по radeon-profile-daemon остались вопросы в посте выше.

(In reply to Koi from comment #16)
> > radeon-profile-daemon
> 
> Эта программа входит в состав монитора radeon-profile, если служба
> отключена, монитор работать не будет. Получается, что программа
> действительно не будет установлена в работоспособном виде. Не все
> пользователи сразу догадаются, что сервис нужно включать вручную. Можно ли
> оставить сервис как есть, или такие случаи не могут быть исключениями? 

Вопрос: что будет, если пакет с сервисом будет установлен, а соответствующего оборудования не будет на машинке?

Я думаю, что оптимальным вариантом в %post при первой установке (но не при обновлении пакета, см. https://www.altlinux.org/SpecTips/triggers) вывести сообщение, что для работы программы необходимо запустить и включить этот сервис. Впрочем, можно ещё поспрашивать мнений.

> Где посмотреть политику сервисов и исключений? Вики нашел Services Policy,
> но там речь о сетевых.

Возможно, что явно оформленной такой политики нет, но мы стараемся, чтобы по-умолчанию было как можно меньше всего включено (из соображения уменьшения поверхности атаки), и в случае необходимости пользователь/администратор явно включал нужную ему функциональность.

(Ответ для Vladimir D. Seleznev на комментарий #18)
>Вопрос: что будет, если пакет с сервисом будет установлен, а соответствующего 
>оборудования не будет на машинке?

Ничего страшного не происходит. Сервис запускается и ждёт клиента.
При запуске клиента radeo-profile выдаётся сообщение, что карт radeon не обнаружено.

Если сервис не запущен, то radeo-profile запускается, ничего не выдаёт. Но при этом ничего и не доступно.

На мой взгляд, так как эта штуковина в дистрибутивы входить не будет ни в каком виде, и в тоже время её никто не вытягивает, то не вижу проблем с включением сервиса при установке пакета.

(In reply to Антон Мидюков from comment #19)
> (Ответ для Vladimir D. Seleznev на комментарий #18)
> >Вопрос: что будет, если пакет с сервисом будет установлен, а соответствующего 
> >оборудования не будет на машинке?
> 
> Ничего страшного не происходит. Сервис запускается и ждёт клиента.
> При запуске клиента radeo-profile выдаётся сообщение, что карт radeon не
> обнаружено.
> 
> Если сервис не запущен, то radeo-profile запускается, ничего не выдаёт. Но
> при этом ничего и не доступно.
> 
> На мой взгляд, так как эта штуковина в дистрибутивы входить не будет ни в
> каком виде, и в тоже время её никто не вытягивает, то не вижу проблем с
> включением сервиса при установке пакета.

Аргумент валидный, однако всё-равно кажется неправильным, что запускается некий процесс, особенно от рута. Наверное, в качестве исключения, особенно если известно, что ничто не вытянет этот пакет, можно оставить, я думаю нужно спросить мнение ещё одного участника.

В целом беглым взглядом по остальным изменениям всё OK.

Я полагаю, кандидат освоится и готов.

(Ответ для Vladimir D. Seleznev на комментарий #20)
> нужно спросить мнение ещё одного участника.
Спасибо, поспрашиваю еще.

Уважаемый секретарь, не пора ли завершить процедуру?

Адрес подписан на devel@.
Пользователь добавлен в группу мейнтейнеров.

Желаю удачного мейнтейнерства!

Спасибо. Буду стараться чтобы не подвести команду.