Bug 40164 - [done] join koi@
Summary: [done] join koi@
Status: CLOSED FIXED
Alias: None
Product: Team Accounts
Classification: Development
Component: join (show other bugs)
Version: unspecified
Hardware: x86_64 Linux
: P5 normal
Assignee: Gleb F-Malinovskiy
QA Contact: Andrey Cherepanov
URL: http://altlinux.org/Team/Join/Secretary
Keywords:
Depends on:
Blocks:
 
Reported: 2021-06-05 09:50 MSK by Koi
Modified: 2022-04-21 01:40 MSK (History)
5 users (show)

See Also:


Attachments
GPG (6.78 KB, text/plain)
2021-06-05 09:50 MSK, Koi
no flags Details
SSH (102 bytes, text/plain)
2021-06-05 09:51 MSK, Koi
no flags Details
GPG1 (3.01 KB, text/plain)
2021-06-30 17:10 MSK, Koi
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Koi 2021-06-05 09:50:53 MSK
Created attachment 9402 [details]
GPG

Имя: koi
Mail: eg.evgeniy@gmail.com
Ментор: antohami@altlinux.org
Цели: Хочу научится собирать пакеты и по возможности помочь чем смогу развиваться любимому дистрибутиву. Еще хотел бы собрать программу crow-translate.
Comment 1 Koi 2021-06-05 09:51:49 MSK
Created attachment 9403 [details]
SSH
Comment 2 Антон Мидюков 2021-06-05 14:05:50 MSK
(Ответ для Koi на комментарий #0)
> Ментор: antohami@altlinux.org

Подтверждаю.
Comment 3 Michael Shigorin 2021-06-10 22:53:09 MSK
Ура! :)
Comment 4 Gleb F-Malinovskiy 2021-06-30 13:47:23 MSK
(Ответ для Koi на комментарий #1)
> Создано вложение 9403 [details] [подробности]
> SSH
Ok.

(Ответ для Koi на комментарий #0)
> Создано вложение 9402 [details] [подробности]
> GPG
В этом файле находится целых два gpg ключа, одного будет вполне достаточно.
Comment 5 Koi 2021-06-30 17:10:20 MSK
Created attachment 9465 [details]
GPG1

Исправленный ключ
Comment 6 Koi 2021-09-06 19:30:09 MSK
Ни хочу никого отвлекать или лишний раз навязываться, но обо мне случайно не забыли? Подтверждение почты отправлено.
Comment 7 Антон Мидюков 2021-09-09 20:49:35 MSK
Когда уже кандидату предоставят доступ к git.alt?
Comment 8 Антон Мидюков 2021-10-18 16:53:52 MSK
Кандидат к git.alt готов. С git умеет работать.
Comment 9 Gleb F-Malinovskiy 2021-10-18 17:06:25 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.
Comment 10 Антон Мидюков 2021-10-20 13:39:29 MSK
Один пакет для сборки готов. Прошу предоставить кандидату доступ к сборочнице.
Comment 11 Gleb F-Malinovskiy 2021-11-29 16:02:44 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.

T/J/S -> 3.4.
Comment 12 Антон Мидюков 2022-01-22 19:30:10 MSK
Кандидат успешно собрал 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/

Считаю, что кандидат готов к самостоятельной сборке пакетов.
Comment 13 Gleb F-Malinovskiy 2022-02-01 12:51:15 MSK
Призван ещё один человек (vseleznv@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.
Comment 14 Vladimir D. Seleznev 2022-02-04 00:24:01 MSK
Общие замечания по собранным пакетам.

По 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/
Comment 15 Koi 2022-02-05 19:21:48 MSK
(Ответ для 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
> 
> Политика для сервисов по-умолчанию (за небольшим исключение) быть
> выключенными. Это нужно исправить.
Понял, исправлю.
Comment 16 Koi 2022-02-22 20:20:20 MSK
> radeon-profile-daemon

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

Где посмотреть политику сервисов и исключений? Вики нашел Services Policy, но там речь о сетевых.
Comment 17 Koi 2022-03-01 19:32:08 MSK
(Ответ для Vladimir D. Seleznev на комментарий #14)
Все исправил, но по radeon-profile-daemon остались вопросы в посте выше.
Comment 18 Vladimir D. Seleznev 2022-03-11 11:26:28 MSK
(In reply to Koi from comment #16)
> > radeon-profile-daemon
> 
> Эта программа входит в состав монитора radeon-profile, если служба
> отключена, монитор работать не будет. Получается, что программа
> действительно не будет установлена в работоспособном виде. Не все
> пользователи сразу догадаются, что сервис нужно включать вручную. Можно ли
> оставить сервис как есть, или такие случаи не могут быть исключениями? 

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

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

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

Возможно, что явно оформленной такой политики нет, но мы стараемся, чтобы по-умолчанию было как можно меньше всего включено (из соображения уменьшения поверхности атаки), и в случае необходимости пользователь/администратор явно включал нужную ему функциональность.
Comment 19 Антон Мидюков 2022-03-11 11:38:44 MSK
(Ответ для Vladimir D. Seleznev на комментарий #18)
>Вопрос: что будет, если пакет с сервисом будет установлен, а соответствующего 
>оборудования не будет на машинке?

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

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

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

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

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

Я полагаю, кандидат освоится и готов.
Comment 21 Koi 2022-03-14 22:27:09 MSK
(Ответ для Vladimir D. Seleznev на комментарий #20)
> нужно спросить мнение ещё одного участника.
Спасибо, поспрашиваю еще.
Comment 22 Антон Мидюков 2022-04-07 10:14:36 MSK
Уважаемый секретарь, не пора ли завершить процедуру?
Comment 23 Gleb F-Malinovskiy 2022-04-19 16:10:00 MSK
Адрес подписан на devel@.
Пользователь добавлен в группу мейнтейнеров.

Желаю удачного мейнтейнерства!
Comment 24 Koi 2022-04-21 00:24:50 MSK
Спасибо. Буду стараться чтобы не подвести команду.