Created attachment 11842 [details]
GPG public key

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

Выдайте, пожалуйста, кандидату гитовницу.

(In reply to Alexander Stepchenko from comment #0)
> Created attachment 11839 [details]
> SSH public key
Ok.
(In reply to Alexander Stepchenko from comment #1)
> Created attachment 11842 [details]
> GPG public key
Ok.

Вперёд!

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

T/J/S -> 2.3.

Выложил 3 пакета tldr, shtab (зависимость tldr) и Discord.

https://git.altlinux.org/people/geochip/packages/

Прошу выдать сборочницу.

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

T/J/S -> 3.5.

Собрал на сборочнице запушенные 3 пакета, получил EPERM:

python3-module-shtab и tldr:
https://git.altlinux.org/tasks/311868/

discord
https://git.altlinux.org/tasks/311871/

(In reply to Alexander Stepchenko from comment #10)
> discord
> https://git.altlinux.org/tasks/311871/

А какая у этих бинарников лицензия?  С пакетом следует распространять лицензию и она, естественно, должна содержать пункт о том, что мы можем это распространять.
Уже попадание такого кода на наш git-хостинг плохо, не говоря про репозиторий.

Уважаемый секретарь, выделите, пожалуйста, моему кандидату рецензента. На мой взгляд, Александр усвоил некоторые базовые представления о сборке в сизиф и способен к самостоятельному анализу и решению проблем при сборке.

(In reply to Gleb F-Malinovskiy from comment #11)
> (In reply to Alexander Stepchenko from comment #10)
> > discord
> > https://git.altlinux.org/tasks/311871/
> 
> А какая у этих бинарников лицензия?  С пакетом следует распространять
> лицензию и она, естественно, должна содержать пункт о том, что мы можем это
> распространять.
> Уже попадание такого кода на наш git-хостинг плохо, не говоря про
> репозиторий.

Для распространения этих бинарников необходимо разрешение. Я ещё с конца декабря несколько раз писал на почту поддержки Discord, чтобы получить разрешение на распространение в нашем репозитории, но мне так и не ответили. Из git-хостинга репозиторий удалил пока что.

Обновил следующие пакеты:

mysql-workbench-community-8.0.32-alt1
https://git.altlinux.org/tasks/314624/

gdal-3.6.2-alt1
https://git.altlinux.org/tasks/314051/

conan-1.58.0-alt1
https://git.altlinux.org/tasks/315213/

python3-module-distro-1.8.0-alt1
https://git.altlinux.org/tasks/315544/

Здравствуйте. Обновляю kubernetes до новой версии, и, видимо, не хватает памяти на гитовнице. Пишет "Disk quota exceeded".
Можно выделить еще 1 ГБ?

Готово.

Поскольку я уже начал смотреть, будем считать, что я ревьюер.

Проблемы:
* tldr и gopass: запакованы каталоги, которые не принадлежат этому пакету
%dir %_datadir/bash-completion
%dir %_datadir/bash-completion/completions
%dir %_datadir/zsh
%dir %_datadir/zsh/site-functions

%dir %_datadir/bash-completion/
%dir %_datadir/bash-completion/completions/
%dir %_datadir/zsh/
%dir %_datadir/zsh/site-functions/
%dir %_datadir/fish/
%dir %_datadir/fish/vendor_completions.d/
это неправильно.
* mysql-workbench-community: Вы обновили сам пакет, а забандленную версию antlr решили почему-то не обновлять, скорее всего это ошибка.
* python3-module-distro: пробовали ли вы отправить ваш патч в апстрим?
* kubernetes: вы заменили KUBE_GIT_COMMIT=%commit на KUBE_GIT_COMMIT=%release.  Я думаю, что плохая идея использовать поле предназначенное для сущности совсем другого типа для нашего release.
* cargo-cache: зачем пакету явная зависимость на /proc, если у бинарного пакета rust уже есть такая зависимость?

Замечания:
* python3-module-shtab: Вместо VERSION_UNKNOWN в качестве заменяемой строки лучше использовать называние типа @MODULE_VERSION@, при виде такой надписи всем сразу понятно, что она должна быть автоматически заменена на значение версии.
* tldr: вместо внутреннего макроса rpm %__python3 лучше просто запускать python3, это на самом деле касается всех внутренних макросов.

Мысли вслух:
* На самом деле, каталог .gear/ ничем не лучше для упаковки spec-а и патчей, чем любой другой каталог, если бы я собирал эти пакеты, я бы использовал каталог alt/.

(In reply to Gleb F-Malinovskiy from comment #17)
> Поскольку я уже начал смотреть, будем считать, что я ревьюер.
> 
> Проблемы:
> * tldr и gopass: запакованы каталоги, которые не принадлежат этому пакету

Это же касается и kubernetes:
%dir %_sysconfdir/systemd
%dir %_sysconfdir/systemd/system.conf.d
%dir %_sysconfdir/systemd/system

Если бы все каталоги из "post-install unowned files" нужно было бы запаковать в пакет, то это был бы не warning.  Это предупреждение, которое значит, что вы могли забыть запаковать каталог, который принадлежит вашему пакету или забыли поставить зависимость на какой-то пакет, которому принадлежит каталог.

tldr:
Запаковка каталогов, которые не принадлежат данному пакету, была исправлена не мною ранее.
https://git.altlinux.org/gears/p/python3-module-tldr.git?p=python3-module-tldr.git;a=blobdiff;f=.gear/tldr.spec;h=3c8853b71828083a8e871b341bd07a8094cda9d0;hp=ceb79b85dc8d443850e72580fb8ed403e8dd1f30;hb=2f8d3fe42262c56ba6abf3cefcfe4800f84979d8;hpb=75ee50da136e4c262e473235ed3309b4e0367abb
Аналогично с %__python3.
https://git.altlinux.org/gears/p/python3-module-tldr.git?p=python3-module-tldr.git;a=commitdiff;h=8fa9b27bd730c22681280346cc06f737bcea6dbe


gopass:
Исправил.
https://git.altlinux.org/tasks/323147/


mysql-workbench-community:
Обновил до последней версии 8.0.33, обновил забандленную antlr до 4.11.1. В версии mysql-workbench-community-8.0.33 необходим libantlr4-runtime.so.4.11.1, поэтому обновил также пакет libantlr4.
https://git.altlinux.org/tasks/320075/

Было замечание от ментора, что предпочтительней было бы избавиться от забандленного antlr-4.11.1-complete.jar, а вместо него использовать упаковынный. В таком случае также необходимо будет обновить пакет antlr4-project. Я подготовил пример того, как это могло бы выглядеть в таске. Так как я не являюсь мэнтейнером ни libantlr4, ни antlr4-project, думаю, что лучше свести их изменения к минимуму, поэтому предпочел первый вариант с забандленный antlr-4.11.1-complete.jar.
https://git.altlinux.org/tasks/324153/


python3-module-distro:
Патч был отправлен в апстрим и успешно замерджен. Ожидается выход следующей версии с включенным патчем.
https://github.com/python-distro/distro/pull/354


kubernetes:
Замена %commit на %release обосновывается двумя причинами.
1. Проще поддерживать пакет, потому что не нужно каждый раз вручную менять хэш коммита в спеке.
2. Вообще хэш коммита используется для идентификации коммита, из которого было собрано ПО. В случае репозитория Sisyphus с помощью значения %release проще и быстрее индентифицировать, из какого коммита было собрано ПО.
Было бы предпочтительней использовать хэш коммита, если бы ПО собиралось, например из последнего коммита ветки master, но в наших репозиториях все программы собираются из релиз тегов. Кроме того, накладываются дополнительные коммиты (добавление патчей, изменения спека и пр.).

В данный момент происходит разбитие пакета kubernetes на пакеты kubernetes1.22, kubernetes1.23, kubernetes1.24, kubernetes1.25 и kubernetes1.26. Причиной является то, что для обновления старых кластеров k8s с версии 1.22 до последней версии в Sisyphus (1.26) необходимо поочередно обновить кластер по всем минорным версиям вверх (1.22 -> 1.23 -> 1.24 -> 1.25 -> 1.26).
Подробнее https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade/

Запаковка каталогов, которые не принадлежат данному пакету в kubernetes будет исправлена в процессе разбиения пакета.


cargo-cache:
Замечание исправлено.
https://git.altlinux.org/tasks/324159/


python3-module-shtab:
Обновлен до последней версии 1.6.2. Замечание исправлено.
https://git.altlinux.org/tasks/324168/

Обновил flannel:
https://git.altlinux.org/tasks/324291/

Здравствуйте. Можно получить обратную связь по обсуждаемым вопросам и процессу джойна?
Есть ли еще какие-нибудь замечания?

Вроде, не было больше замечаний.
Хорошая работа.

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

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