Bug 50400

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

Пользователь добавлен в группу мейнтейнеров.

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