Bug 54552

Created attachment 18634 [details]
Прикрепляю публичную часть SSH ключа

Created attachment 18635 [details]
Прикрепляю публичную часть GPG ключа

Ключи выглядят правдоподобно, нужно подтверждение ментора.

(Ответ для Gleb F-Malinovskiy на комментарий #3)
> Ключи выглядят правдоподобно, нужно подтверждение ментора.

> Имя ментора: Евгений Синельников (sin@altlinux.org)
Я за него. Менторство подтверждаю.

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

Кандидат собрал несколько пакетов и предложил их мне на ревью.

Общие замечания:
1. В каждый пакет желательно добавлять `%define _unpackaged_files_terminate_build 1` в начало spec-файла. Это поможет заметить файлы, которые забыли запаковать в пакет.

https://github.com/OlgaKamaeva/PrettyErrors/commits/sisyphus/
1. Имя пакета не соответствует имени с Pypi.

https://github.com/OlgaKamaeva/chaospy/commits/sisyphus/
1. Вместе с пакетом не собрана документация.

https://github.com/OlgaKamaeva/numpoly/commits/sisyphus/
Пока вопросов нет.

В целом, выглядит хорошо, можно пробовать собирать на сборочнице.

Следующим шагом предлагаю попробовать собрать что-нибудь на Python с использованием https://www.altlinux.org/Management_of_Python_dependencies_sources

glebfm@, прошу перевести кандидата на третий этап.

(Ответ для Andrey Limachko на комментарий #6)
> Кандидат собрал несколько пакетов и предложил их мне на ревью.
> 
> Общие замечания:
> 1. В каждый пакет желательно добавлять `%define
> _unpackaged_files_terminate_build 1` в начало spec-файла. Это поможет
> заметить файлы, которые забыли запаковать в пакет.
Желательно не добавлять этот мусор. Я устал вычищать пакеты от этих строчек. Неслепой мейнтейнер обязательно заметит строчку про unowned files в конце лога.

> Следующим шагом предлагаю попробовать собрать что-нибудь на Python с
> использованием
> https://www.altlinux.org/Management_of_Python_dependencies_sources
Как мейнтейнер Python3 и ответственный за всё это безобразие, рекомендую не тратить своё время на изучение вредной схемы, которая в скором времени будет полностью устранена из репозитория.

2mentor: Андрей, сосредоточьтесь, пожалуйста на обучении кандидата важным и полезным навыкам. Вредить они научатся как-то сами=)

Так же, прошу кандидата использовать при сборке rpm-build-python3, в котором определены используемые в спеке макросы. Пакет rpm-build-pyproject в сборке не участвует. Это важно и ваш ментор эту разницу видимо не понимает.

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

(In reply to Grigory Ustinov from comment #7)
> > 1. В каждый пакет желательно добавлять `%define
> > _unpackaged_files_terminate_build 1` в начало spec-файла. Это поможет
> > заметить файлы, которые забыли запаковать в пакет.
> Желательно не добавлять этот мусор. Я устал вычищать пакеты от этих строчек.
> Неслепой мейнтейнер обязательно заметит строчку про unowned files в конце
> лога.
Тут я как мейнтейнер категорически не согласен (особенно, с вычисткой) -- новые файлы могут появиться в любой момент, в том числе на другой архитектуре или при пересборке пакета.  Если бы от этого не ломалось так много пакетов, этот режим был бы включён по умолчанию.

(In reply to Stanislav Levin from comment #9)
> Может быть, советы будет давать ментор?
Моё мнение, что советы может давать кто угодно из team.  В необходимые умения майнтейнера входит понимание того, какие советы хорошие, а какие не очень.

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

T/J/S -> 2.3.

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

T/J/S -> 3.6.

(Ответ для Grigory Ustinov на комментарий #8)
> Так же, прошу кандидата использовать при сборке rpm-build-python3, в котором
> определены используемые в спеке макросы. Пакет rpm-build-pyproject в сборке
> не участвует. Это важно и ваш ментор эту разницу видимо не понимает.

Действительно. Только сейчас обратил внимание. Спасибо!

olkmv@, прошу исправить.

(Ответ для Grigory Ustinov на комментарий #7)
> (Ответ для Andrey Limachko на комментарий #6)
> > Кандидат собрал несколько пакетов и предложил их мне на ревью.
> > 
> > Общие замечания:
> > 1. В каждый пакет желательно добавлять `%define
> > _unpackaged_files_terminate_build 1` в начало spec-файла. Это поможет
> > заметить файлы, которые забыли запаковать в пакет.
> Желательно не добавлять этот мусор. Я устал вычищать пакеты от этих строчек.
> Неслепой мейнтейнер обязательно заметит строчку про unowned files в конце
> лога.
> 
> > Следующим шагом предлагаю попробовать собрать что-нибудь на Python с
> > использованием
> > https://www.altlinux.org/Management_of_Python_dependencies_sources
> Как мейнтейнер Python3 и ответственный за всё это безобразие, рекомендую не
> тратить своё время на изучение вредной схемы, которая в скором времени будет
> полностью устранена из репозитория.
> 
> 2mentor: Андрей, сосредоточьтесь, пожалуйста на обучении кандидата важным и
> полезным навыкам. Вредить они научатся как-то сами=)

Спасибо за рекомендацию. На данный момент же факты таковы:
$ grep -r "%pyproject_runtimedeps_metadata" | wc -l
828

Считаю, что кандидату необходимо, как минимум, ознакомиться с этой схемой.

(In reply to Gleb F-Malinovskiy from comment #10)
> (In reply to Grigory Ustinov from comment #7)
> > > 1. В каждый пакет желательно добавлять `%define
> > > _unpackaged_files_terminate_build 1` в начало spec-файла. Это поможет
> > > заметить файлы, которые забыли запаковать в пакет.
> > Желательно не добавлять этот мусор. Я устал вычищать пакеты от этих строчек.
> > Неслепой мейнтейнер обязательно заметит строчку про unowned files в конце
> > лога.
> Тут я как мейнтейнер категорически не согласен (особенно, с вычисткой) --
> новые файлы могут появиться в любой момент, в том числе на другой
> архитектуре или при пересборке пакета.  Если бы от этого не ломалось так
> много пакетов, этот режим был бы включён по умолчанию.

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

Sisyphus/x86_64/latest/success$ ls |wc -l
21139
Sisyphus/x86_64/latest/success$ grep -Flx 'warning: Installed (but unpackaged) file(s) found:' * |wc -l
966