Псевдоним: fiersik Почта для пересылки: fiersik.kouji@alt-gnome.ru Имя ментора: Антон Мидюков Почта ментора: antohami@altlinux.org Цель вступления: Создание и поддержка регулярной сборки на основе оконного менеджера Hyprland, сборка соответствующих экосистеме пакетов. Например hyprpaper, hypridle и hyprlock.
Created attachment 16159 [details] ssh key Добавлен ключ ssh.
Created attachment 16160 [details] gpg key Добавлен ключ gpg.
Подтверждаю, что согласился быть ментором. Псевдоним fiersik свободен. Ключи выглядят нормально. Прошу предоставить кандидату доступ к git.alt
Comment on attachment 16159 [details] ssh key Кандидат мне написал, что собирается заменить ssh ключ.
Created attachment 16190 [details] ssh key Новый SSH ключ
(Ответ для Кирилл Уницаев на комментарий #5) > Создано вложение 16190 [details] [подробности] > ssh key > > Новый SSH ключ Выглядит нормально. Не теряйте больше ключики. Прошу предоставить кандидату доступ к git.alt и сборочнице. Пакеты собирать он умеет.
Ментор есть, ключи в порядке.
ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3.
Прошу предоставить кандидату доступ к сборочнице также. Локально пакеты он собирать умеет.
ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. Адрес подписан на devel@. T/J/S -> 3.6.
Собраны пакеты: Уже в Сизифе: hyprland-legacyrenderer ALT-repo: https://git.altlinux.org/people/fiersik/packages/hyprland-legacyrenderer.git URL: https://github.com/hyprwm/Hyprland hypridle ALT-repo: https://git.altlinux.org/people/fiersik/packages/hypridle.git URL: https://github.com/hyprwm/hypridle hyprpaper ALT-repo: https://git.altlinux.org/people/fiersik/packages/hyprpaper.git URL: https://github.com/hyprwm/hyprpaper hyprpicker ALT-repo: https://git.altlinux.org/people/fiersik/packages/hyprpicker.git URL: https://github.com/hyprwm/hyprpicker libhyprutils ALT-repo: https://git.altlinux.org/people/fiersik/packages/libhyprutils.git URL: https://github.com/hyprwm/hyprutils hyprshot ALT-repo: https://git.altlinux.org/people/fiersik/packages/hyprshot.git URL: https://github.com/Gustash/Hyprshot Попадут в репозиторий завтра: mpvpaper ALT-repo: https://git.altlinux.org/people/fiersik/packages/mpvpaper.git URL: https://github.com/GhostNaN/mpvpaper libgtk4-layer-shell ALT-repo: https://git.altlinux.org/people/fiersik/packages/libgtk4-layer-shell.git URL: https://github.com/wmww/gtk4-layer-shell
Считаю, что кандидат готов к самостоятельной сборке пакетов. Прошу назначить рецензента.
Призван рецензент (egori@) для независимой оценки готовности кандидата. T/J/S -> 4.2.
Добрый вечер, Хотел бы извиниться за задержку с рецензией. Кандидат успел собрать довольно много пакетов к стадии 4.2, 40 если быть точным, поэтому мне потребуется больше времени, чем я изначально предполагал. На данный момент я уже посмотрел бОльшую часть пакетов, постараюсь закончить ревью до нового года.
Мы никуда не торопимся :) Однако, перед продолжением, попрошу обратить внимание на багу #50853
Провел ревью Кандидата. Общие замечания для многих пакетов: - Отсутствует точка в конце записи changelog, которая начинается с большой буквы [1]. - Формат релизного коммита отличается от стандартного (gear-commit) -- например [2][3]. Прошу обратить внимание на эти стилистические замечания при сборке пакетов в дальнейшем. Замечания по пакетам: - hyprclock: Стоит также упаковать .conf .desktop README.md LICENSE файлы. - nwg-dock-hyprland: Стоит также упаковать config/style.css README.md LICENSE файлы. - ironbar: Стоит также упаковать target/completions/* README.md LICENSE файлы. - nwg-dock-hyprland: Явный Requires на libgtk-layer-shell libgtk+3 не нужен, AutoReq вроде бы справляется сам. - eww: Аналогично (libgtk-layer-shell libgtk+3). - polkit-hyprland: Аналогично (libqt6-quickcontrols2). - hyprland-qtutils: Аналогично (libqt6-quickcontrols2). - hyprsysteminfo: Аналогично (libqt6-quickcontrols2). - eww: Не соблюдение FHS, /etc используется для файлов конфигурации, examples/eww-bar там не должно быть. Прошу исправить эти замечания. Также хочу обратить внимание Кандидата на список пакетов с более свежими версиями: https://packages.altlinux.org/en/sisyphus/maintainers/fiersik/watch По результатам ревью: + Кандидат продемонстрировал сборку из tarball и из апстримного тега. + Кандидат продемонстрировал сборку с помощью meson, cmake. + Кандидат продемонстрировал сборку rust, c++, golang пакетов. + Кандидат продемонстрировал сборку шрифтов. + Кандидат продемонстрировал vendoring (rust, golang). + Кандидат продемонстрировал упаковку системных сервисов. + Кандидат продемонстрировал соблюдение Shared Libs Policy. + Кандидат продемонстрировал поддержку (обновление) своих пакетов. - Кандидат не продемонстрировал сборку python пакетов. - Кандидат не продемонстрировал сборку с autotools. Решение: На данный момент Кандидат собрал уже 46 пакетов в сизиф и является активным мантейнером подсистемы hyprland. Несмотря на то, что Кандидат не собрал python и autotools пакетов, думаю с его текущем опытом это не должно вызвать трудностей. Общее качество пакетов на достаточно высоком уровне. Отдельное спасибо Ментору за проделанную работу. Считаю, что Кандидат готов к самостоятельной работе, и после исправления вышеуказанных замечаний его можно переводить на следующий этап (принять в ALT Linux Team). [1] https://www.altlinux.org/%D0%A0%D1%83%D0%BA%D0%BE%D0%B2%D0%BE%D0%B4%D1%81%D1%82%D0%B2%D0%BE_%D0%BF%D0%BE_%D0%BD%D0%B0%D0%BF%D0%B8%D1%81%D0%B0%D0%BD%D0%B8%D1%8E_changelog [2] https://git.altlinux.org/gears/m/mpvpaper.git?p=mpvpaper.git;a=commit;h=969b446495621528858097a1bf154e2b34a01c18 [3] https://git.altlinux.org/gears/h/hyprpicker.git?p=hyprpicker.git;a=commit;h=257af16b53b3484ef1acbbea577a098c0a3f778b
(Ответ для Egor Ignatov на комментарий #16) > Провел ревью Кандидата. Огромное спасибо за столь большую работу ) > Общие замечания для многих пакетов: > - Отсутствует точка в конце записи changelog, которая начинается с большой > буквы [1]. > - Формат релизного коммита отличается от стандартного (gear-commit) -- > например [2][3]. > Прошу обратить внимание на эти стилистические замечания при сборке пакетов в > дальнейшем. Учту! > Замечания по пакетам: > - hyprclock: Стоит также упаковать .conf .desktop README.md LICENSE файлы. > - nwg-dock-hyprland: Стоит также упаковать config/style.css README.md > LICENSE файлы. > - ironbar: Стоит также упаковать target/completions/* README.md LICENSE > файлы. Пересоберу с исправлениями в ближайшее время. nwg-dock-hyprland ведёт себя слишком нестабильно, если не получится исправить совместно с апстримом - пакет будет удалён. > - nwg-dock-hyprland: Явный Requires на libgtk-layer-shell libgtk+3 не нужен, > AutoReq вроде бы справляется сам. > - eww: Аналогично (libgtk-layer-shell libgtk+3). > - polkit-hyprland: Аналогично (libqt6-quickcontrols2). > - hyprland-qtutils: Аналогично (libqt6-quickcontrols2). > - hyprsysteminfo: Аналогично (libqt6-quickcontrols2). Пересобирал их несколько раз, по какой-то причине зависимости не подтягиваются. Попробую ещё, может где-то ошибся. > - eww: Не соблюдение FHS, /etc используется для файлов конфигурации, > examples/eww-bar там не должно быть. > > Прошу исправить эти замечания. eww-bar - это пример базовой конфигурации, он подтягивается при использовании. > Также хочу обратить внимание Кандидата на список пакетов с более свежими > версиями: > https://packages.altlinux.org/en/sisyphus/maintainers/fiersik/watch Обновляю по мере возможности: nwg-dock-hyprland - есть таски с новыми версиями, но они работают очень плохо, а чаще и вовсе не работают. entt-devel - в новой версии есть ошибка, жду следующего релиза. hypridle - обновление невозможно (https://bugzilla.altlinux.org/51838) Остальные пакеты уже обновлены. > По результатам ревью: > + Кандидат продемонстрировал сборку из tarball и из апстримного тега. > + Кандидат продемонстрировал сборку с помощью meson, cmake. > + Кандидат продемонстрировал сборку rust, c++, golang пакетов. > + Кандидат продемонстрировал сборку шрифтов. > + Кандидат продемонстрировал vendoring (rust, golang). > + Кандидат продемонстрировал упаковку системных сервисов. > + Кандидат продемонстрировал соблюдение Shared Libs Policy. > + Кандидат продемонстрировал поддержку (обновление) своих пакетов. > - Кандидат не продемонстрировал сборку python пакетов. > - Кандидат не продемонстрировал сборку с autotools. Всё впереди, пока не было необходимости. > Решение: > ... > Считаю, что Кандидат готов к самостоятельной работе, и после исправления > вышеуказанных замечаний его можно переводить на следующий этап (принять в > ALT Linux Team). Ещё раз огромное спасибо!
(In reply to Egor Ignatov from comment #16) > Провел ревью Кандидата. > > Общие замечания для многих пакетов: > - Отсутствует точка в конце записи changelog, которая начинается с большой > буквы [1]. Это очень спорный момент. Никогда не ставлю точку в ChangeLog в конце записи. Вкусовщину на вики привели.
(Ответ для Egor Ignatov на комментарий #16) > Замечания по пакетам: > - hyprclock: Стоит также упаковать .conf .desktop README.md LICENSE файлы. - .desktop в апстриме неправильный - LICENSE нету в теге, только в main его обновлять пока не стал, позже в апстрим писать буду. > - nwg-dock-hyprland: Стоит также упаковать config/style.css README.md > LICENSE файлы. - config/style.css уже есть в пакете - README.md и LICENSE упаковал > - ironbar: Стоит также упаковать target/completions/* README.md LICENSE > файлы. - target/completions/* будет только в следующей версии, сейчас упаковывать нечего - README.md и LICENSE упаковал > - nwg-dock-hyprland: Явный Requires на libgtk-layer-shell libgtk+3 не нужен > - eww: Аналогично (libgtk-layer-shell libgtk+3). > - polkit-hyprland: Аналогично (libqt6-quickcontrols2). > - hyprland-qtutils: Аналогично (libqt6-quickcontrols2). > - hyprsysteminfo: Аналогично (libqt6-quickcontrols2). исправлено > - eww: Не соблюдение FHS, /etc используется для файлов конфигурации, > examples/eww-bar там не должно быть. Теперь /usr/share/doc/eww-../eww-bar/* Все изменения в таске https://packages.altlinux.org/ru/tasks/371007/
(In reply to Кирилл Уницаев from comment #19) > (Ответ для Egor Ignatov на комментарий #16) > > Замечания по пакетам: > > - hyprclock: Стоит также упаковать .conf .desktop README.md LICENSE файлы. > > - .desktop в апстриме неправильный > - LICENSE нету в теге, только в main Да, вижу. > его обновлять пока не стал, позже в апстрим писать буду. Тут речь про .desktop файл видимо, если да, то ОК. > > - nwg-dock-hyprland: Стоит также упаковать config/style.css README.md > > LICENSE файлы. > > - config/style.css уже есть в пакете > - README.md и LICENSE упаковал ОК. > > - ironbar: Стоит также упаковать target/completions/* README.md LICENSE > > файлы. > > - target/completions/* будет только в следующей версии, сейчас упаковывать > нечего > - README.md и LICENSE упаковал ОК. > > - nwg-dock-hyprland: Явный Requires на libgtk-layer-shell libgtk+3 не нужен > > - eww: Аналогично (libgtk-layer-shell libgtk+3). > > - polkit-hyprland: Аналогично (libqt6-quickcontrols2). > > - hyprland-qtutils: Аналогично (libqt6-quickcontrols2). > > - hyprsysteminfo: Аналогично (libqt6-quickcontrols2). > > исправлено ОК. > > - eww: Не соблюдение FHS, /etc используется для файлов конфигурации, > > examples/eww-bar там не должно быть. > > Теперь /usr/share/doc/eww-../eww-bar/* ОК. > > Все изменения в таске https://packages.altlinux.org/ru/tasks/371007/ Задание заапрувил. Все нужные исправления сделаны. Кандидат готов отправлять пакеты в Сизиф. Желаю удачного мейнтейнерства!
Последнее замечание, только сейчас заметил. Если вы изменяете запись в changelog, а потом амендите релизный коммит, то делать это нужно через 'gear-commit --amend' чтобы commit message тоже обновился. https://git.altlinux.org/tasks/371007/gears/100/git?p=git;a=commit;h=04ba23e09c5e013cd9c110dc99098bf56faa9b71
Пользователь добавлен в группу мейнтейнеров. Желаю удачного мейнтейнерства!
