Created attachment 16319 [details]
gpg key

Псевдоним       : sheriffkorov
Почта           : Kirill Sharov <sheriffkorov@altlinux.org>
Пересылка почты : xiencol@gmail.com
Имя ментора     : Михаил Чернигин
Почта ментора   : chernigin@altlinux.org
Моя цель        : Научиться собирать пакеты

Менторство подтверждаю

Ключи в порядке, что-то мешает двигаться дальше?

Ментор есть, ключи в порядке.
T/J/S -> 1.3.

Думаю, можем двигаться дальше.

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

T/J/S -> 2.3.

Научился работать с git.alt.

Пакет 1: https://git.altlinux.org/people/sheriffkorov/packages/smlnj.git
Пакет 2: https://git.altlinux.org/people/sheriffkorov/packages/alterator-backend-systeminfo.git

Жду подтверждения ментора.

Отлично, тогда переходим к шагу с gyle.

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

T/J/S -> 3.6.

Доброго времени суток!

Научился работать со сборочницей.
Собрал 4 пакета:
1. workflow (https://git.altlinux.org/people/sheriffkorov/packages/workflow.git) — C++ библиотека;
2. wfrest (https://git.altlinux.org/people/sheriffkorov/packages/wfrest.git) — C++ библиотека;
3. alterator-backend-systeminfo (https://git.altlinux.org/people/sheriffkorov/packages/alterator-backend-systeminfo.git) — noarch (непосредственный разработчик);
4. alterator-application-systeminfo (https://git.altlinux.org/people/sheriffkorov/packages/alterator-application-systeminfo.git) — графическое Qt/C++ приложение (непосредственный разработчик).

Логи задач сборочницы:
Задача 361063 (пакеты 1-2): https://git.altlinux.org/tasks/361063/logs/events.5.1.log
Задача 361235 (пакеты 3-4): https://git.altlinux.org/tasks/361235/logs/events.1.1.log

Жду слова ментора. :)

Собрал ещё один пакет.
jujutsu (https://git.altlinux.org/people/sheriffkorov/packages/jujutsu.git) — написанная на Rust утилита VCS.

В задаче 362804: https://git.altlinux.org/tasks/362804/logs/events.1.1.log


Также пересобрал задачи 361063 и 361235.
Лог 361063: https://git.altlinux.org/tasks/361063/logs/events.7.1.log
Лог 361235: https://git.altlinux.org/tasks/361235/logs/events.2.1.log

Да, мне вроде бы всё нравится. Давайте звать рецензента!

Обновил пакет ccache до 4.10 (в sisyphus версия 4.6).

С мейнтейнером пакета в процессе установки связи: жду ответа.

ccache (https://git.altlinux.org/people/sheriffkorov/packages/ccache.git).

Также новая версия ccache повлекла за собой новые зависимости. Одна из зависимостей отсутствует в sisyphus, поэтому её тоже пришлось "опакетить".

span-lite (https://git.altlinux.org/people/sheriffkorov/packages/span-lite.git) — header-only библиотека C++.

В задаче 365333: https://git.altlinux.org/tasks/365333/logs/events.3.1.log

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

T/J/S -> 4.2.

Прошелся по всем имеющимся на данный момент сборочным заданиям Кандидата.

================================
https://packages.altlinux.org/en/tasks/361063/

workflow
Собрана библиотека на C++(Cmake)
Замечания:
- Пакет имеет более актуальную версию в апстриме.
- Лицензия указана не по SPDX.
- Релизный коммит создан не через gear-commit.
- Не соответствует Shared Libs Policy.

wfrest
Собран фреймворк на C++(Cmake)
Замечания:
- Пакет имеет более актуальную версию в апстриме.
- Релизный коммит создан не через gear-commit.
- Бинарный пакет должен быть -devel пакетом.
- Странная строчка в спеке '%exclude %_docdir/%name-0.9.6/README.md'
  Во-первых, не понятно зачем исключать README.md. Во-вторых, захардкоженная
  версия уже поменялась.
- Было бы неплохо еще упакетить README.md, LICENSE.

================================
https://packages.altlinux.org/en/tasks/362804/

jujutsu
Собран пакет на rust.
Замечания:
- Пакет имеет более актуальную версию в апстриме.
- Релизный коммит создан не через gear-commit.
- Неправильный порядок коммитов: вендоринг должен быть до релизного коммита.
- При сборке 'warning: Installed (but unpackaged) file(s) found'
  Можно использовать '%define _unpackaged_files_terminate_build 1' чтобы
  отслеживать такой мусор.
- Было бы неплохо еще упакетить README.md, LICENSE.

================================
https://packages.altlinux.org/en/tasks/365333/

span-lite
Собрана single-file header-only C++ библиотека.
Замечания:
- Релизный коммит создан не через gear-commit.

ccache
Обновлен чужой пакет.
Замечания:
- Пакет имеет более актуальную версию в апстриме.
- Релизный коммит создан не через gear-commit.
- Неправильный порядок коммитов: релизный коммит должен быть последним.
- Длина строки в description уж очень сильно ограничена (видимо 41 символ).
- Чтобы каждый раз не добавлять/убирать 'copy: .gear/*.patch' из gear/rules
  можно использовать 'copy?: .gear/*.patch'.
- При сборке чужих пакетов стоит стараться соблюдать стиль владельца пакета.
  Например, в данном случае в changelog было бы уместнее написать
  "new version 4.10" вместо "New version 4.10."

================================
https://packages.altlinux.org/en/tasks/361235/

alterator-application-systeminfo
- Каша из коммитов:
  + Изменения связанные с пакетированием (спек, .gear/rules, т.д.) должны быть в
    отдельных от исходников коммитах.
  + Выпускающий (релизный) коммит, который содержит спек, должен быть последним.
  + Релизный коммит должен быть создан утилитой gear-commit.
  **Как Ментор пропустил такое в сизиф - загадка**
- Лишний 'Requires: libqbase' AutoReq сам справляется.

alterator-backend-systeminfo
- Аналогичная каша из коммитов

===============================

Помимо явных проблем с использованием git и gear-commit, в целом спеки выглядят
неплохо. Базовые навыки сборки на достаточном уровне, теперь от Кандидата
хотелось бы увидеть понимание и применение остальных, более продвинутых,
инструментов и технологий, а также большего разнообразия в системах
сборки/языках программирования собираемых пакетов.

Что уже продемонстрировал Кандидат:
- Базовые навыки сборки rpm-пакетов.
- Сборку из апстримного тега.
- Сборку C++ пакетов с помощью Cmake.
- Сборку rust пакета.
- Вендоринг rust зависимостей.
- Сборку noarch пакета.

Что хотелось бы видеть от кандидата:
- Сопровождение собственных пакетов (обновление до следующей версии).
- Умение работать с чужими/осиротевшими пакетами.
- Умение прикладывать патчи и/или делать коммиты к апстримным исходникам.
- Включение тестов при сборке пакетов.
- Понимание и применение Shared Libs Policy.
- Опыт использование других систем сборки, например: make, autotools, meson.
- Опыт сборки пакетов на других языках, например: C, python, golang, bash.
- Опыт использования pre/post скриптов (triggers).
- Опыт использование макросов условной сборки %if_with/%if_enable/%ifarch.
- Использование '%define _unpackaged_files_terminate_build 1'.
- Использование '%config(noreplace)' для упаковки конфигурационных файлов.
- Использование 'gear-remotes' для сохранения указателя на апстрим.
- Сборку системных сервисов (демонов).

================================
Решение:

Кандидат пока не готов к самостоятельной сборке пакетов в сизиф.

================================
В связи с переводом Кандидата на стадию 4.0 при наличии таких серьезных и при
этом очевидных ошибок, как тут[1][2][3] с порядком и содержанием коммитов,
хотелось бы высказать свои сомнения в готовности Ментора быть ментором.

Первое, что бросилось в глаза - Ментор совершенно не поддерживает свои пакеты
(из 13 пакетов, собранных в sisyphus, 6 требуют обновления
https://packages.altlinux.org/en/sisyphus/maintainers/chernigin/watch).

Видно, что Ментор не является активным мейнтейнером, он занимается только
пакетами подсистемы alterator, и с момента прохождения join другие свои пакеты
практически не трогал.

Поэтому считаю, что Ментор, хоть и является членом Team, не имеет достаточного
опыта, чтобы быть ментором.

[1] https://git.altlinux.org/people/sheriffkorov/packages/?p=ccache.git;a=log;h=359bc3e83791fd3c500be924954dca2c4137424a
[2] https://git.altlinux.org/people/sheriffkorov/packages/?p=alterator-application-systeminfo.git;a=log;h=182bfc73c93cffdb325f45002ee560935497ca21
[3] https://git.altlinux.org/people/sheriffkorov/packages/?p=alterator-backend-systeminfo.git;a=log;h=a63aac080e26d7c35dbf104b5ed2e96d8001844a

Здравствуйте, Егор! Премного благодарю за уделённое время и ценную обратную связь! 

Данные Вами фактические замечания учёл. Хочу доложить о текущем статусе пакетов.

1. Пересобрал workflow (библиотека С++, CMake) и wfrest (фреймворк C++, CMake).
Помимо учтённых замечаний по пакету:
- включил тесты;
- приложил патчи;
- опакетил LICENSE и README;
- использовал %define _unpackaged_files_terminate_build 1;
- использовал gear-remotes-save.

Два патча касаются wfrest, один из которых отключает тесты на утечки памяти, так как возникли проблемы с запуском valgrind в chroot, которые не смог решить. Такое решение, возможно, временное.

В сборочном задании 361063 (статус EPERM).

2. Пересобрал alterator-backend-systeminfo (noarch, bash, make) и alterator-application-systeminfo (Qt/C++, CMake).
Учёл замечания как мог. Являюсь разработчиком исходников, поэтому делал, как было принято. Но мне тоже это не нравится, теперь веду всё, что связано с пакетированием и релизами в sisyphus, в отдельной ветке.
Помимо учтённых замечаний:
- опакетил LICENSE и CHANGELOG;
- использовал %define _unpackaged_files_terminate_build 1;
- использовал gear-remotes-save.

Пакеты alterator-backend-systeminfo и alterator-application-systeminfo с актуальными версиями находятся в Сизифе.

В планах закончить пересборку jujutsu (Rust, cargo) и ccache (C++, CMake) с учётом замечаний и пожеланий.

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

(In reply to Egor Ignatov from comment #16)
> Прошелся по всем имеющимся на данный момент сборочным заданиям Кандидата.
> 
> ================================
> https://packages.altlinux.org/en/tasks/361063/
> 
> workflow
> Собрана библиотека на C++(Cmake)
> Замечания:
> - Пакет имеет более актуальную версию в апстриме.
> - Лицензия указана не по SPDX.
> - Релизный коммит создан не через gear-commit.
> - Не соответствует Shared Libs Policy.
> 
> wfrest
> Собран фреймворк на C++(Cmake)
> Замечания:
> - Пакет имеет более актуальную версию в апстриме.
> - Релизный коммит создан не через gear-commit.
> - Бинарный пакет должен быть -devel пакетом.
> - Странная строчка в спеке '%exclude %_docdir/%name-0.9.6/README.md'
>   Во-первых, не понятно зачем исключать README.md. Во-вторых, захардкоженная
>   версия уже поменялась.
> - Было бы неплохо еще упакетить README.md, LICENSE.
> 
> ================================
> https://packages.altlinux.org/en/tasks/362804/
> 
> jujutsu
> Собран пакет на rust.
> Замечания:
> - Пакет имеет более актуальную версию в апстриме.
> - Релизный коммит создан не через gear-commit.
> - Неправильный порядок коммитов: вендоринг должен быть до релизного коммита.
> - При сборке 'warning: Installed (but unpackaged) file(s) found'
>   Можно использовать '%define _unpackaged_files_terminate_build 1' чтобы
>   отслеживать такой мусор.
> - Было бы неплохо еще упакетить README.md, LICENSE.
> 
> ================================
> https://packages.altlinux.org/en/tasks/365333/
> 
> span-lite
> Собрана single-file header-only C++ библиотека.
> Замечания:
> - Релизный коммит создан не через gear-commit.
> 
> ccache
> Обновлен чужой пакет.
> Замечания:
> - Пакет имеет более актуальную версию в апстриме.
> - Релизный коммит создан не через gear-commit.
> - Неправильный порядок коммитов: релизный коммит должен быть последним.
> - Длина строки в description уж очень сильно ограничена (видимо 41 символ).
> - Чтобы каждый раз не добавлять/убирать 'copy: .gear/*.patch' из gear/rules
>   можно использовать 'copy?: .gear/*.patch'.
> - При сборке чужих пакетов стоит стараться соблюдать стиль владельца пакета.
>   Например, в данном случае в changelog было бы уместнее написать
>   "new version 4.10" вместо "New version 4.10."
> 
> ================================
> https://packages.altlinux.org/en/tasks/361235/
> 
> alterator-application-systeminfo
> - Каша из коммитов:
>   + Изменения связанные с пакетированием (спек, .gear/rules, т.д.) должны
> быть в
>     отдельных от исходников коммитах.
>   + Выпускающий (релизный) коммит, который содержит спек, должен быть
> последним.
>   + Релизный коммит должен быть создан утилитой gear-commit.
>   **Как Ментор пропустил такое в сизиф - загадка**
> - Лишний 'Requires: libqbase' AutoReq сам справляется.
> 
> alterator-backend-systeminfo
> - Аналогичная каша из коммитов
> 
> ===============================
> 
> Помимо явных проблем с использованием git и gear-commit, в целом спеки
> выглядят
> неплохо. Базовые навыки сборки на достаточном уровне, теперь от Кандидата
> хотелось бы увидеть понимание и применение остальных, более продвинутых,
> инструментов и технологий, а также большего разнообразия в системах
> сборки/языках программирования собираемых пакетов.
> 
> Что уже продемонстрировал Кандидат:
> - Базовые навыки сборки rpm-пакетов.
> - Сборку из апстримного тега.
> - Сборку C++ пакетов с помощью Cmake.
> - Сборку rust пакета.
> - Вендоринг rust зависимостей.
> - Сборку noarch пакета.
> 
> Что хотелось бы видеть от кандидата:
> - Сопровождение собственных пакетов (обновление до следующей версии).
> - Умение работать с чужими/осиротевшими пакетами.
> - Умение прикладывать патчи и/или делать коммиты к апстримным исходникам.
> - Включение тестов при сборке пакетов.
> - Понимание и применение Shared Libs Policy.
> - Опыт использование других систем сборки, например: make, autotools, meson.
> - Опыт сборки пакетов на других языках, например: C, python, golang, bash.
> - Опыт использования pre/post скриптов (triggers).
> - Опыт использование макросов условной сборки %if_with/%if_enable/%ifarch.
> - Использование '%define _unpackaged_files_terminate_build 1'.
> - Использование '%config(noreplace)' для упаковки конфигурационных файлов.
> - Использование 'gear-remotes' для сохранения указателя на апстрим.
> - Сборку системных сервисов (демонов).
> 
> ================================
> Решение:
> 
> Кандидат пока не готов к самостоятельной сборке пакетов в сизиф.
> 
> ================================
> В связи с переводом Кандидата на стадию 4.0 при наличии таких серьезных и при
> этом очевидных ошибок, как тут[1][2][3] с порядком и содержанием коммитов,
> хотелось бы высказать свои сомнения в готовности Ментора быть ментором.
> 
> Первое, что бросилось в глаза - Ментор совершенно не поддерживает свои пакеты
> (из 13 пакетов, собранных в sisyphus, 6 требуют обновления
> https://packages.altlinux.org/en/sisyphus/maintainers/chernigin/watch).
> 
> Видно, что Ментор не является активным мейнтейнером, он занимается только
> пакетами подсистемы alterator, и с момента прохождения join другие свои
> пакеты
> практически не трогал.
> 
> Поэтому считаю, что Ментор, хоть и является членом Team, не имеет
> достаточного
> опыта, чтобы быть ментором.
> 
> [1]
> https://git.altlinux.org/people/sheriffkorov/packages/?p=ccache.git;a=log;
> h=359bc3e83791fd3c500be924954dca2c4137424a
> [2]
> https://git.altlinux.org/people/sheriffkorov/packages/?p=alterator-
> application-systeminfo.git;a=log;h=182bfc73c93cffdb325f45002ee560935497ca21
> [3]
> https://git.altlinux.org/people/sheriffkorov/packages/?p=alterator-backend-
> systeminfo.git;a=log;h=a63aac080e26d7c35dbf104b5ed2e96d8001844a