Created attachment 9542 [details]
gpg public key

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

Кандидат, для разминки, планирует опакетить для Sisyphus [1] и [2].

[1] https://github.com/MaskRay/ccls
[2] https://github.com/iptraf-ng/iptraf-ng

(In reply to ancieg from comment #0)
> Created attachment 9541 [details]
> ssh public key
(In reply to ancieg from comment #1)
> Created attachment 9542 [details]
> gpg public key

Ok.

T/J/S -> 1.3.

Небольшая корректировка информации.

Почта для пересылки теперь: ancieg@bk.ru

Считаю, что кандидат готов перейти к следующему шагу

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

T/J/S -> 2.3.

(Ответ для Gleb F-Malinovskiy на комментарий #6)
> ssh ключ на gitery.alt зарегистрирован.
> Адрес для пересылки создан.
> 
> T/J/S -> 2.3.

Прошу прощения за долгий ответ.

Прошу изменить почтовый адрес для пересылки на тот, что указан в comment #4.

Считаю, что кандидат готов к переходу к 3 этапу.

(Ответ для Anton Zhukharev на комментарий #7)
> (Ответ для Gleb F-Malinovskiy на комментарий #6)
> > ssh ключ на gitery.alt зарегистрирован.
> > Адрес для пересылки создан.
> > 
> > T/J/S -> 2.3.
> 
> Прошу прощения за долгий ответ.
> 
> Прошу изменить почтовый адрес для пересылки на тот, что указан в comment #4.

Решился ли вопрос с адресом пересылки?

(In reply to Николай Костригин from comment #9)
> Решился ли вопрос с адресом пересылки?

Да.

Пакет alt-gpgkeys обновлён.

T/J/S -> 3.5.

(Ответ для Gleb F-Malinovskiy на комментарий #11)
> Пакет alt-gpgkeys обновлён.
> 
> T/J/S -> 3.5.

Пункт 3.4 из T/J/S выполнен?
Доступа к gyle.atlinux.org по ssh нет с таким сообщением:

    ssh: alt_ancieg@gyle.altlinux.org: Permission denied (publickey).

(In reply to Anton Zhukharev from comment #12)
> (Ответ для Gleb F-Malinovskiy на комментарий #11)
> > Пакет alt-gpgkeys обновлён.
> > 
> > T/J/S -> 3.5.
> 
> Пункт 3.4 из T/J/S выполнен?
> Доступа к gyle.atlinux.org по ssh нет с таким сообщением:
> 
>     ssh: alt_ancieg@gyle.altlinux.org: Permission denied (publickey).

Прошу прощения, это была ошибка с моей стороны, попробуйте ещё раз.

(Ответ для Gleb F-Malinovskiy на комментарий #13)
> (In reply to Anton Zhukharev from comment #12)
> > (Ответ для Gleb F-Malinovskiy на комментарий #11)
> > > Пакет alt-gpgkeys обновлён.
> > > 
> > > T/J/S -> 3.5.
> > 
> > Пункт 3.4 из T/J/S выполнен?
> > Доступа к gyle.atlinux.org по ssh нет с таким сообщением:
> > 
> >     ssh: alt_ancieg@gyle.altlinux.org: Permission denied (publickey).
> 
> Прошу прощения, это была ошибка с моей стороны, попробуйте ещё раз.

Проверил, теперь работает корректно. Спасибо!

Результаты сборок на данный момент.


Собранные задания со статусом [TESTED]:
  ccls - C++ language server.
  [sisyphus]
    * 299321 ccls-0.20210330-alt1 (статическая линковка)
    * 299324 ccls-0.20210330-alt2 (динамическая линковка из-за
      нового LLVM 13.0)
  [p10]
    * 299326 ccls-0.20210330-alt1 (статическая линковка)

  iptraf-ng - a console-based network monitoring program for Linux.
  [sisyphus]
    * 299322 iptraf-ng-1.2.1-alt1

  age - a simple, modern and secure encryption tool.
  [sisyphus]
    * 299504 age-1.0.0-alt3
  [p10]
    * 299506 age-1.0.0-alt3

  libcbor - a C library for parsing and generating CBOR,
  the general-purpose schema-less binary data format.
  [sisyphus]
    * 299505 - libcbor-0.9.0-alt1



Провалившиеся задания со статусом [FAILED]:
  age - a simple, modern and secure encryption tool.
  [sisyphus]
    * 299388 age-1.0.0-alt1
        на архитектурах i586 и armh возникла ошибка со следующим
        сообщением: "ld: cannot find -lc"
    * 299402 age-1.0.0-atl2
        из-за явно выставленной в спеке BuildArch: x86_64 произошла
        ошибка сборки на остальных архитектурах

(Ответ для Anton Zhukharev на комментарий #15)
>     * 299388 age-1.0.0-alt1
>         на архитектурах i586 и armh возникла ошибка со следующим
>         сообщением: "ld: cannot find -lc"

  А glibc-devel-static есть в BuildRequires?

>     * 299402 age-1.0.0-atl2
>         из-за явно выставленной в спеке BuildArch: x86_64 произошла
>         ошибка сборки на остальных архитектурах

  Вы хотели сказать, наверное: "ExclusiveArch: x86_64"?

(Ответ для Andrew Vasilyev на комментарий #16)
> (Ответ для Anton Zhukharev на комментарий #15)
> >     * 299388 age-1.0.0-alt1
> >         на архитектурах i586 и armh возникла ошибка со следующим
> >         сообщением: "ld: cannot find -lc"
> 
>   А glibc-devel-static есть в BuildRequires?

С тэга 1.0.0-alt3 в BuildRequires есть glibc

> >     * 299402 age-1.0.0-atl2
> >         из-за явно выставленной в спеке BuildArch: x86_64 произошла
> >         ошибка сборки на остальных архитектурах
> 
>   Вы хотели сказать, наверное: "ExclusiveArch: x86_64"?

Нет, я написал верно :)
Было именно "BuildArch: x86_64", что приводило к ошибке вида:

    error: No compatible architectures found for build

на архитектурах i586, aarch64, ppc64le и armh.


С тэга 1.0.0-alt3 выставлено "ExclusiveArch: %go_arches"

В iptraf-ng упакована конфигурация logrotate (также добавлен макрос %set_verify_elf_method strict):

  299687 iptraf-ng-1.2.1-alt2 [sisyphus] - TESTED
  299691 iptraf-ng-1.2.1-alt2 [p10]      - TESTED


Cобран новый пакет yggdrasil (https://yggdrasil-network.github.io/).
Yggdrasil is an overlay network implementation of a new routing scheme for
mesh networks. It is designed to be a future-proof decentralised alternative
to the structured routing protocols commonly used today on the Internet and 
other networks.
Для запуска необходимо запустить systemd-сервис yggdrasil.service.

  299694 yggdrasil-0.4.3-alt4 [sisyphus] - TESTED
  299695 yggdrasil-0.4.3-alt4 [p10]      - TESTED

Собран пакет python3-module-openssh-wrapper-ArcticaProject для Sisyphus.
Это форк уже существующего Python3-модуля, который уже опакечен в
python3-module-openssh-wrapper.
В этом форке добавленны несколько новых возможностей (на мой взгляд
удобных), которые мне нужны в своём проекте на Python 3.

  300000 python3-module-openssh-wrapper-ArcticaProject-0.5-alt2 - TESTED

Собран новый пакет pyotherside в Sisyphus.
A Qt plugin providing access to a Python 3 interpreter from QML for creating asynchronous mobile and Desktop UIs with Python.

    300032 pyotherside.git 1.5.9-alt2 - TESTED.

Предлагаю призвать рецензента для оценки уровня подготовки кандидата.

(Ответ для Anton Zhukharev на комментарий #20)
> Собран новый пакет pyotherside в Sisyphus.
> A Qt plugin providing access to a Python 3 interpreter from QML for creating
> asynchronous mobile and Desktop UIs with Python.
> 
>     300032 pyotherside.git 1.5.9-alt2 - TESTED.

Собрана новая версия пакета pyotherside в Sisyphus.
Новая версия включает ранее выключенные тесты (которые были отключены из-за
опасений, что они не проходят на архитектурах, отличных от i586 и x86_64).
Однако, они не проходили только на архитектуре armh. Поэтому с новой версии
автотесты включены для всех архитектур, кроме проблемной armh.

    300094 pyotherside.git 1.5.9-alt3 - TESTED.

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

T/J/S -> 4.2.

В целом всё хорошо.

Общее замечание - в Summary: в конце точку ставить нет необходимости.

Ещё одно общее замечание-просьба: сохраняйте информацию об git апстрима:
gear-remotes-save
чтобы можно было восстановить upstream командой:
gear-remotes-restore

Пакет pyotherside
Удалять в спеке файлы/каталоги лучше без опции -f, так как иначе скорее всего эта строчка в спеке останется навсегда:
https://git.altlinux.org/tasks/300092/gears/100/git?p=git;a=blob;f=.gear/pyotherside.spec;h=91274d0a5d4fbeff4c5f8813dc1237b1d7394d02;hb=447b60cb2e58a135980c6f159029a95793302c29#l43

Вместо armh лучше использовать макрос %arm:
https://git.altlinux.org/tasks/300092/gears/100/git?p=git;a=blob;f=.gear/pyotherside.spec;h=91274d0a5d4fbeff4c5f8813dc1237b1d7394d02;hb=447b60cb2e58a135980c6f159029a95793302c29#l3

Пакет python3-module-openssh-wrapper
Описание коммита не соответствует изменению:
https://git.altlinux.org/tasks/300000/gears/100/git?p=git;a=commitdiff;h=7f48024decacdbf53687ad82c13d734c5a2092e6
Кроме того, здесь явно требуется объяснение в многострочном описании коммита зачем провайдесу указывается версия %EVR. Я так понимаю, чтобы вес был другой, чем у пакета python3-module-openssh-wrapper.

Пакет yggdrasil
В preun не стоит удалять группу, так как файлы останутся с непонятным GID.
Но, если сильно хочется, добавьте условие if [ $1 -eq 0 ]; then, чтобы удаление группы происходило только при удалении пакета, а не при его обновлении.
https://git.altlinux.org/tasks/299694/gears/100/git?p=git;a=blob;f=.gear/yggdrasil.spec;h=5b515b15e01acbde47296fcf3b488f672125537d;hb=6028c31b10fc153a97081c1fc7fc86ae74467a1f#l60

Пакет iptraf-ng
Не нужно прописывать в Buildrequires gcc и glibc, так как они входят в базовый сборочный чрут:
https://git.altlinux.org/tasks/299687/gears/100/git?p=git;a=blob;f=.gear/iptraf-ng.spec;h=8967952efd2cd1156aec95db92893e67d296e34c;hb=a7a06039646bebd34a4a67214392cdad4dec82b5#l17

Пакет age
Не нужно прописывать в Buildrequires glibc, так как он входит в базовый сборочный чрут:
https://git.altlinux.org/tasks/299504/gears/100/git?p=git;a=blob;f=.gear/age.spec;h=1b497cf74471645a78633c47d678156df42285fb;hb=b2f786bd308bbc9337863620c48f4805edd42e86#l17
Странный коммит, в котором делается не только то, что в его описании:
https://git.altlinux.org/tasks/299504/gears/100/git?p=git;a=commitdiff;h=3fc39937f8733f384632e5b093aa5d698ff05b08

Пакет libcbor
Опять gcc в BuildRequires:
https://git.altlinux.org/tasks/299505/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=57847cb6f6e2333a4dc90fe69d17dcff1cdd558d;hb=4d17dd666988d32e5d424ea4c8aee5086262b5b5#l16

rpm-macros-* нужно помещать в BuildRequires(pre):
https://git.altlinux.org/tasks/299505/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=57847cb6f6e2333a4dc90fe69d17dcff1cdd558d;hb=4d17dd666988d32e5d424ea4c8aee5086262b5b5#l14

Вот тут очень интересно:
https://git.altlinux.org/tasks/299505/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=57847cb6f6e2333a4dc90fe69d17dcff1cdd558d;hb=4d17dd666988d32e5d424ea4c8aee5086262b5b5#l46
pushd .
   cd doc
   make SPHINXBUILD="sphinx-build-3" BUILDDIR=%_builddir/%name-%version/ man
popd

Почему бы не сделать pushd doc?

man'ы сжимать не надо. Всё само делается.
https://git.altlinux.org/tasks/299505/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=57847cb6f6e2333a4dc90fe69d17dcff1cdd558d;hb=4d17dd666988d32e5d424ea4c8aee5086262b5b5#l54

Пакет ccls
А зачем в p10 собираете со статической линковкой? Какие-то плюсы от этого есть?

(Ответ для Антон Мидюков на комментарий #24)

> Пакет pyotherside
> Удалять в спеке файлы/каталоги лучше без опции -f, так как иначе скорее
> всего эта строчка в спеке останется навсегда
Поправлено (теперь точечно удаляется исполняемый файл для автотестов):
https://git.altlinux.org/tasks/300409/gears/100/git?p=git;a=blob;f=.gear/pyotherside.spec;h=1543525be5b08e7bcc81143d5eb126ee5c35a80b;hb=e6f3962cf6917fa29c86b418d2a16d919a5fc390#l43
Но хотелось бы подробнее узнать почему эта строчка может остаться в спеке навсегда.

> Вместо armh лучше использовать макрос %arm
Поправлено:
https://git.altlinux.org/tasks/300409/gears/100/git?p=git;a=blob;f=.gear/pyotherside.spec;h=1543525be5b08e7bcc81143d5eb126ee5c35a80b;hb=e6f3962cf6917fa29c86b418d2a16d919a5fc390#l3


> Пакет python3-module-openssh-wrapper
> Описание коммита не соответствует изменению: ...
Моя вина, не углядел.

> Кроме того, здесь явно требуется объяснение в многострочном описании коммита
> зачем провайдесу указывается версия %EVR. Я так понимаю, чтобы вес был
> другой, чем у пакета python3-module-openssh-wrapper.
Если не указывать EVR, то сборочница ругается на дубликаты Provides:
https://git.altlinux.org/tasks/300506/logs/events.1.1.log

Поэтому %EVR было выставлено на основании спек-файлов других конфликтующих между
собой пакетов (за пример были взяты спеки для mysql и mariadb).
На данный момент убрал эпоху на основании советов здесь:
https://www.altlinux.org/Spec#Provides
Потому теперь выглядит так:
https://git.altlinux.org/tasks/300585/gears/100/git?p=git;a=blobdiff;f=.gear/python3-module-openssh-wrapper-ArcticaProject.spec;h=ecde1d27c570b1a45117b6aaf1fddeb9266f5345;hp=ef8ded2487f4e57e6f270d340dc5f5ff90e228bd;hb=46c6ff62d468c24168a84cb9d55a4c265de8664a;hpb=1f97bac4cc5e8eb964c451ec872ee295aa0ee708

Данный Python3-пакет является форком для Python3-пакета в python3-module-openssh-wrapper.
Форк имеет такую же версию, как и апстрим, но в нём есть несколько исправленией и улучшений.
Совместимость между форком и апстримом сохранена.


> Пакет yggdrasil
> В preun не стоит удалять группу, так как файлы останутся с непонятным GID.
Хороший аргумент. Я отказался от решения удалять группу:
https://git.altlinux.org/tasks/300415/gears/100/git?p=git;a=blob;f=.gear/yggdrasil.spec;h=fd2962582f88128547d04900a9344bc5124373f8;hb=3da84290418d448e06c80d4b3db8ee8d81f740c3#l59

> Но, если сильно хочется, добавьте условие if [ $1 -eq 0 ]; then, чтобы
> удаление группы происходило только при удалении пакета, а не при его
> обновлении.
Возьму на вооружение.


> Пакет iptraf-ng
> Не нужно прописывать в Buildrequires gcc и glibc, так как они входят в
> базовый сборочный чрут:
В самом последне коммите на ветке Sisyphus эта избыточность удалена:
https://git.altlinux.org/tasks/300402/gears/100/git?p=git;a=blob;f=.gear/iptraf-ng.spec;h=21a6f6b48759d0c3c3316b894ae355c8b3629ddd;hb=74f775ec8a9af5a3b0753403c31e9b8298aacbdb#l17


> Пакет age
> Не нужно прописывать в Buildrequires glibc, так как он входит в базовый
> сборочный чрут
Убрал:
https://git.altlinux.org/tasks/300401/gears/100/git?p=git;a=blob;f=.gear/age.spec;h=2f4ab108a2a5caee0248210858380f4f145b7d46;hb=13f783c6c9cbe464fb95e68248bc98ae04cf95c9#l16

> Странный коммит, в котором делается не только то, что в его описании: ...
К сожалению, снова не углядел.


> Пакет libcbor
> Опять gcc в BuildRequires:
В более всежей версии зависимость от gcc убрана:

> rpm-macros-* нужно помещать в BuildRequires(pre):
> https://git.altlinux.org/tasks/299505/gears/100/git?p=git;a=blob;f=.gear/
> libcbor.spec;h=57847cb6f6e2333a4dc90fe69d17dcff1cdd558d;
> hb=4d17dd666988d32e5d424ea4c8aee5086262b5b5#l14
В более всежей версии rpm-macros-cmake помещён в BuildRequires(pre):
https://git.altlinux.org/tasks/300404/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=a8d9e3a1769459e37a86b45dec3d8a23f84ade95;hb=a5fb04431e4fca84c65a71900847c04f944da677#l14

> Вот тут очень интересно: ...
> pushd .
>    cd doc
>    make SPHINXBUILD="sphinx-build-3" BUILDDIR=%_builddir/%name-%version/ man
> popd
> 
> Почему бы не сделать pushd doc?
Спек давно был написан и там было ещё что-то (уже точно не помню что: коммиты засквошены),
но оказавшись ненужным, было удалено, а структура осталась). Поправлено:
https://git.altlinux.org/tasks/300404/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=a8d9e3a1769459e37a86b45dec3d8a23f84ade95;hb=a5fb04431e4fca84c65a71900847c04f944da677#l44

> man'ы сжимать не надо. Всё само делается.
Дело в том, что спек для libcbor создавался ещё в том году, поэтому я
о возможности автокомпрессии man'ов узнал позже. Сейчас выглядит так:
https://git.altlinux.org/tasks/300404/gears/100/git?p=git;a=blob;f=.gear/libcbor.spec;h=a8d9e3a1769459e37a86b45dec3d8a23f84ade95;hb=a5fb04431e4fca84c65a71900847c04f944da677#l49


> Пакет ccls
> А зачем в p10 собираете со статической линковкой? Какие-то плюсы от этого
> есть?
До какого-то момента большой плюс таки был.

До LLVM 13.0 динамическая линковка не работала совсем и приходилось
использовать статическую (ошибка: https://bugzilla.altlinux.org/42249).

С выходом LLVM 13.0 ошибка исправлена и сборка ccls теперь происходит
с динамической линковкой (я указал в сообщении две сборки к тому же
из-за того, что ccls я собирал и в Sisyphus и в P10, поэтому решил
таким явным образом подчеркнуть, что LLVM 13.0 нет в P10 и нужно туда
собирать с статической линковкой).

(Ответ для Anton Zhukharev на комментарий #25)
> (Ответ для Антон Мидюков на комментарий #24)
> > Пакет pyotherside
> > Удалять в спеке файлы/каталоги лучше без опции -f, так как иначе скорее
> > всего эта строчка в спеке останется навсегда
> Поправлено (теперь точечно удаляется исполняемый файл для автотестов):
> https://git.altlinux.org/tasks/300409/gears/100/git?p=git;a=blob;f=.gear/
> pyotherside.spec;h=1543525be5b08e7bcc81143d5eb126ee5c35a80b;
> hb=e6f3962cf6917fa29c86b418d2a16d919a5fc390#l43
> Но хотелось бы подробнее узнать почему эта строчка может остаться в спеке
> навсегда.
> 

Вы меня не поняли. Когда ставится опция -f, то сборка не упадёт, когда этого файла уже не будет существовать. Т.е. с опцией -f будет успешно удаляться пустое место. Это замечание не требовало исправления. Хотел на этот факт обратить внимание, вот и всё.

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

(Ответ для Антон Мидюков на комментарий #26)
> (Ответ для Anton Zhukharev на комментарий #25)
> > (Ответ для Антон Мидюков на комментарий #24)
> > > Пакет pyotherside
> > > Удалять в спеке файлы/каталоги лучше без опции -f, так как иначе скорее
> > > всего эта строчка в спеке останется навсегда
> > Поправлено (теперь точечно удаляется исполняемый файл для автотестов):
> > https://git.altlinux.org/tasks/300409/gears/100/git?p=git;a=blob;f=.gear/
> > pyotherside.spec;h=1543525be5b08e7bcc81143d5eb126ee5c35a80b;
> > hb=e6f3962cf6917fa29c86b418d2a16d919a5fc390#l43
> > Но хотелось бы подробнее узнать почему эта строчка может остаться в спеке
> > навсегда.
> > 
> 
> Вы меня не поняли. Когда ставится опция -f, то сборка не упадёт, когда этого
> файла уже не будет существовать. Т.е. с опцией -f будет успешно удаляться
> пустое место. Это замечание не требовало исправления. Хотел на этот факт
> обратить внимание, вот и всё.
Спасибо за ответ, теперь понятно :)

> По остальному тогда всё в порядке.
> Кандидат готов к самостоятельной сборке пакетов.
Пора менять топик в соответствии с https://www.altlinux.org/Team/Join/Reviewer.

Адрес подписан на devel@.
Пользователь добавлен в группу мейнтейнеров.

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