Bug 56287

Created attachment 19704 [details]
ssh public key

Подтверждаю.

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

Кандидат готов начать вступление.

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

T/J/S -> 2.3.

Подопечный готов собирать пакеты.

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

T/J/S -> 3.6.

Подопечный готов собирать пакеты.

Призываю рецензента для независимой оценки готовности кандидата.

Я вижу что задания аппрувит не ментор. ментор поменялся ? если так, то лучше он бы посмотрел на пакеты и сказал готов ли кандидат или нет к самостоятельной работе.

(Ответ для Anton Farygin на комментарий #9)
> Я вижу что задания аппрувит не ментор. ментор поменялся ? если так, то лучше
> он бы посмотрел на пакеты и сказал готов ли кандидат или нет к
> самостоятельной работе.

Он на этапе 4.0. Ментор уже не нужен. Ждёт рецензента.

ментор нужен на любом этапе

Новый ментор: Айрат Махмутов (rauty)

(Ответ для Pavel Petrykin на комментарий #12)
> Новый ментор: Айрат Махмутов (rauty)

Подтверждаю

(In reply to protvin@altlinux.org from comment #8)
> Подопечный готов собирать пакеты.
> 
> Призываю рецензента для независимой оценки готовности кандидата.
Допустим, это буду я.

Уважаемые менторы и кандидаты! При переводе на стадию, которая нынче называется T/J/S 4.0 (по ссылке), указывайте, куда смотреть на подготовленные кандидатом пакеты. В нынешней ситуации, где люди по полгода в очереди ждут, а их пакеты бесконтрольно уходят в сизиф (видели мы прецеденты), особенно непонятно.

Я пока начал смотреть iwd, который мне случайно (не шучу) примелькался. :)

(In reply to Arseny Maslennikov from comment #14)
> (In reply to protvin@altlinux.org from comment #8)
> > Подопечный готов собирать пакеты.
> Я пока начал смотреть iwd, который мне случайно (не шучу) примелькался. :)
Из мелочёвки: для %_prefix/lib/systemd/system у нас есть макрос: %_unitdir (и, соотв, %_userunitdir для %_prefix/lib/systemd/user).

В каталоге с исходниками:

  % git grep UseDefaultI
  .gear/dist/main.conf:UseDefaultInterface=true
  src/iwd.config.rst:   * - UseDefaultInterface (**deprecated**)
  src/manager.c:                          "UseDefaultInterface", &use_default))
  src/manager.c:          l_warn("[General].UseDefaultInterface is deprecated, please "

Прямо в текущей версии исходников уже написали, что не рекомендуется к использованию этот параметр. Зачем же вы его взяли? Какой параметр стоит всё же использовать, если вдруг драйвер у радиомодуля кривой?

Кроме вышеперечисленного и bug 59562 (который не относится к упаковке напрямую* всё же), iwd упакован прилично. Воспользуюсь возможностью задать вопрос кандидату: как вы планируете обновлять пакет по мере выхода новых версий, что делать с gear-репозиторием?

____
* Если считать, что там правильно поступили с 80-iwd.link и конфигом по умолчанию, то пакет оформлен грамотно и ясно; мои заявления по поводу этого бага о другом.

(In reply to Arseny Maslennikov from comment #14)
> Я пока начал смотреть iwd, который мне случайно (не шучу) примелькался. :)
Ещё из мелочёвки: я б вместо %_datadir/man/man1/* последним компонентом ставил *.1*, чтобы был номер секции явно. Это позволит ловить за апстримом ошибки, где ман из секции 8 ставят в каталог для 1 и т. п.

Добрый день!

(In reply to Arseny Maslennikov from comment #15)
> Из мелочёвки: для %_prefix/lib/systemd/system у нас есть макрос: %_unitdir
> (и, соотв, %_userunitdir для %_prefix/lib/systemd/user).
Подставил %_unitdir в соответствующие пути.

(In reply to Arseny Maslennikov from comment #15)
> Зачем же вы его взяли? Какой параметр стоит всё же использовать, если вдруг
> драйвер у радиомодуля кривой?
Последовал устаревшему совету. Современный параметр - [DriverQuirks].DefaultInterface.

(In reply to Arseny Maslennikov from comment #15)
> Воспользуюсь возможностью задать вопрос кандидату: как вы планируете обновлять
> пакет по мере выхода новых версий, что делать с gear-репозиторием?
В целом стандартно:
- Загрузить апстримную ветку
- Cмержить версию до которой обновляем в ветку sisyphus (Опционально, но удобно для мейнтейнера)
- Обновить из какого тега идет сборка и поднять версию в спеке
- Подправить тело спека если есть изменения в процессе сборки, новые файлы для установки, и т.п.
Нюанс только в необходимости следить за актуальностью завендоренных header'ов ell.

(In reply to Arseny Maslennikov from comment #16)
> Ещё из мелочёвки: я б вместо %_datadir/man/man1/* последним компонентом
> ставил *.1*, чтобы был номер секции явно. Это позволит ловить за апстримом
> ошибки, где ман из секции 8 ставят в каталог для 1 и т. п.
Добавил явно-указанный номер секции. Также подставил %_man<x>dir в соответствующие пути.

Ссылка на таск с изменениями:
https://packages.altlinux.org/en/tasks/421642/

(In reply to Pavel Petrykin from comment #17)
> - Cмержить версию до которой обновляем в ветку sisyphus (Опционально, но
> удобно для мейнтейнера)
Неточно выразился, перефразирую:
Иметь коммит из которого собираем в истории обязательно; Оставлять апстримные исходники в ветке sisyphus опционально.

(In reply to Pavel Petrykin from comment #18)
> (In reply to Pavel Petrykin from comment #17)
> > - Cмержить версию до которой обновляем в ветку sisyphus (Опционально, но
> > удобно для мейнтейнера)
> Неточно выразился, перефразирую:
> Иметь коммит из которого собираем в истории обязательно; Оставлять
> апстримные исходники в ветке sisyphus опционально.
Здесь, да, сложились разные подходы, применяемые в зависимости от пакета и по вкусу мейнтейнера.

(In reply to Pavel Petrykin from comment #17)
> (In reply to Arseny Maslennikov from comment #16)
> > Ещё из мелочёвки: я б вместо %_datadir/man/man1/* последним компонентом
> > ставил *.1*, чтобы был номер секции явно. Это позволит ловить за апстримом
> > ошибки, где ман из секции 8 ставят в каталог для 1 и т. п.
> Добавил явно-указанный номер секции. Также подставил %_man<x>dir в
> соответствующие пути.
Очень хорошо, что обратили внимание на %_man1dir! я вот почему-то не обратил, как будто у вас в спеке уже использовался %_man1dir.

> Ссылка на таск с изменениями:
> https://packages.altlinux.org/en/tasks/421642/
Замечательно. К iwd вопросов скорее больше нет.

Всё ещё повторяю призыв.
(In reply to Arseny Maslennikov from comment #14)
> Уважаемые менторы и кандидаты! При переводе на стадию, которая нынче
> называется T/J/S 4.0 (по ссылке), указывайте, куда смотреть на
> подготовленные кандидатом пакеты. В нынешней ситуации, где люди по полгода в
> очереди ждут, а их пакеты бесконтрольно уходят в сизиф (видели мы
> прецеденты), особенно непонятно.
Цель вступления в Team Павел тоже не изложил; то, что по этому поводу сообщает кандидат, в общем, влияет на ревью.

Подразумеваемый ответ "да просто все done/eperm таски посмотрите" — это очень смелый ответ, ранние задания могут не отражать актуальные умения кандидата, но до них непременно докопаются (случаи — без моего участия — были)
В частности, я жду, по крайней мере, ещё две отличные от autotools сборочные системы и что-нибудь ноарчовое.

(Ответ для Arseny Maslennikov на комментарий #20)
> Всё ещё повторяю призыв.
> (In reply to Arseny Maslennikov from comment #14)
> > Уважаемые менторы и кандидаты! При переводе на стадию, которая нынче
> > называется T/J/S 4.0 (по ссылке), указывайте, куда смотреть на
> > подготовленные кандидатом пакеты. В нынешней ситуации, где люди по полгода в
> > очереди ждут, а их пакеты бесконтрольно уходят в сизиф (видели мы
> > прецеденты), особенно непонятно.
> Цель вступления в Team Павел тоже не изложил; то, что по этому поводу
> сообщает кандидат, в общем, влияет на ревью.
> 
> Подразумеваемый ответ "да просто все done/eperm таски посмотрите" — это
> очень смелый ответ, ранние задания могут не отражать актуальные умения
> кандидата, но до них непременно докопаются (случаи — без моего участия —
> были)
> В частности, я жду, по крайней мере, ещё две отличные от autotools сборочные
> системы и что-нибудь ноарчовое.

Посмотрел работы Павла.
Честно, как ментор, я не считаю, что он готов к самостоятельной сборке пакетов.
Есть много косяков в пакетах, таски которых я сам апрувил.
На 4-ый этап его переводил protvin@, который уже не является активным мейнтейнером.

(In reply to Arseny Maslennikov from comment #19)
> Замечательно. К iwd вопросов скорее больше нет.
Нашлось ещё замечание к gear-репозиторию.
Если вы обращаетесь к релизному тегу от upstream, и на gitery выгружаете апстримный коммит, на котором стоит релизный тег — рекомендую пушить ещё и сам тег. Когда задание попадает в репозиторий, теги, смотрящие на входящие коммиты, тоже оказываются в репозитории git.altlinux.org/gears/*/*, что очень удобно читателю.

(In reply to Ajrat Makhmutov from comment #21)
> (Ответ для Arseny Maslennikov на комментарий #20)
> > (In reply to Arseny Maslennikov from comment #14)
> > > указывайте, куда смотреть на
> > > подготовленные кандидатом пакеты. В нынешней ситуации, где люди по полгода в
> > > очереди ждут, а их пакеты бесконтрольно уходят в сизиф (видели мы
> > > прецеденты), особенно непонятно.
> > > 
> > > Подразумеваемый ответ "да просто все done/eperm таски посмотрите" — это
> > > очень смелый ответ, ранние задания могут не отражать актуальные умения
> > > кандидата, но до них непременно докопаются (случаи — без моего участия —
> > > были)
> > > В частности, я жду, по крайней мере, ещё две отличные от autotools сборочные
> > > системы и что-нибудь ноарчовое.
> 
> Посмотрел работы Павла.
> Честно, как ментор, я не считаю, что он готов к самостоятельной сборке
> пакетов.
> Есть много косяков в пакетах, таски которых я сам апрувил.
> На 4-ый этап его переводил protvin@, который уже не является активным
> мейнтейнером.
Айрат, приветствую!
Если уверены и планируете дальше менторствовать, тогда переводите на 3.6.

(Ответ для Arseny Maslennikov на комментарий #23)
> Айрат, приветствую!
> Если уверены и планируете дальше менторствовать, тогда переводите на 3.6.

Перевожу на этап 3.6 в связи с тем, что silverducks@ не готов собирать пакеты самостоятельно.

Окей, желаю удачи; Павел, готовьтесь.
Я не отказываюсь от дальнейшего рецензирования; как будете готовы идти дальше — ставьте сразу 4.2 и пингуйте. :)