Bug 53279

Created attachment 17861 [details]
public gpg

(Ответ для Ivan Khanas на комментарий #0)
> Ментор: Айрат Махмутов ( rauty@altlinux.org )

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

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

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

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

T/J/S -> 2.3.

Created attachment 17972 [details]
новый ключ

Утерян пароль для старого gpg ключа.

(In reply to Ivan Khanas from comment #6)
> Created attachment 17972 [details]
> новый ключ
Выложите ещё ещё и на git.alt, пожалуйста.

Кандидат освоил инструменты сборки и собрал несколько пакетов локально. Так что считаю, что он готов собирать пакеты на сборочнице.
Прошу перенести кандидата на следующий этап

(Ответ для Gleb F-Malinovskiy на комментарий #7)
> (In reply to Ivan Khanas from comment #6)
> > Created attachment 17972 [details] [подробности] [details]
> > новый ключ
> Выложите ещё ещё и на git.alt, пожалуйста.

новый ключ
https://git.altlinux.org/people/xeno/packages/?p=alt-gpgkeys.git;a=summary

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

T/J/S -> 3.6.

Кандидат готов отправлять пакеты в Сизиф.
Прошу призвать рецензента для независимой оценки готовности кандидата.

На данный момент кандидат собрал пакеты:
https://packages.altlinux.org/ru/sisyphus/srpms/dump_syms/
https://packages.altlinux.org/ru/sisyphus/srpms/OmniGraph/
https://packages.altlinux.org/ru/sisyphus/srpms/python3-module-pdoc3/
https://packages.altlinux.org/ru/sisyphus/srpms/gosec/
https://packages.altlinux.org/ru/sisyphus/srpms/redmine-cli/
https://git.altlinux.org/people/xeno/packages/?p=libpostal.git&a=summary

Собрал gradle:
https://packages.altlinux.org/ru/sisyphus/srpms/gradle/
https://packages.altlinux.org/ru/sisyphus/srpms/linstor/

А также обновил пакеты:
https://packages.altlinux.org/ru/sisyphus/srpms/python3-module-h5netcdf/
https://packages.altlinux.org/ru/sisyphus/srpms/python3-module-netCDF4/
https://git.altlinux.org/people/xeno/packages/?p=libvalhalla.git&a=summary
https://git.altlinux.org/people/xeno/packages/?p=osmscout-server.git&a=summary

Коллеги здравствуйте, мне сегодня сообщили, что ранее собранные мною для
join'а пакеты были отправлены в Сизиф. Почему со мной никто не связался по
этому поводу?

$ ssh gitery find-package osmscout-server
> /people/xeno/packages/osmscout-server.git	1749490075
> /people/ved/packages/osmscout-server.git	1730121478

Если возникла острая нужда в оффлайн картах, то давайте вместе будем над
ними работать.

(Ответ для Egor Shestakov на комментарий #12)
> Коллеги здравствуйте, мне сегодня сообщили, что ранее собранные мною для
> join'а пакеты были отправлены в Сизиф. Почему со мной никто не связался по
> этому поводу?
> 
> $ ssh gitery find-package osmscout-server
> > /people/xeno/packages/osmscout-server.git	1749490075
> > /people/ved/packages/osmscout-server.git	1730121478
> 
> Если возникла острая нужда в оффлайн картах, то давайте вместе будем над
> ними работать.

Привет! Я начал собирать pure-maps, не зная, что вы этим занимаетесь. Собрал libpostal и libvalhalla, посмотрел в issue на pure-maps, что upstream предлагает использовать osmscout-server, и только после этого заметил, что pure-maps и osmscout-server висят у вас в заданиях, но еще не в репозитории. Те пакеты, что я собрал до этого я решил отправить,а всё остальное, что связано с pure-maps не трогать и оставить вам, т.к вы потратили время чтобы это сделать, поэтому приношу свои извинения если доставил неприятности этой ситуацией.

Ладно, только вы допустили одну очень серьезную ошибку: обновили удаленную
из Сизифа libvalhalla с помощью другой библиотеки с таким же названием, но
предназначенной для абсолютно других вещей. Первая libvalhalla -- это
библиотека для выкачивания метаданных из медиафайлов
(http://libvalhalla.geexbox.org), вторая libvalhalla -- это библиотека для
построенния маршрутов (http://valhalla.github.io/valhalla/). Я увидел эту коллизию
и собрал её как libvalhalla-router. Сейчас надо хотя бы как-то отметить эту
ситуацию, потому что рано или поздно кому-то первая библиотека понадобится
в Сизифе.

(Ответ для Egor Shestakov на комментарий #14)
> Ладно, только вы допустили одну очень серьезную ошибку: обновили удаленную
> из Сизифа libvalhalla с помощью другой библиотеки с таким же названием, но
> предназначенной для абсолютно других вещей. Первая libvalhalla -- это
> библиотека для выкачивания метаданных из медиафайлов
> (http://libvalhalla.geexbox.org), вторая libvalhalla -- это библиотека для
> построенния маршрутов (http://valhalla.github.io/valhalla/). Я увидел эту
> коллизию
> и собрал её как libvalhalla-router. Сейчас надо хотя бы как-то отметить эту
> ситуацию, потому что рано или поздно кому-то первая библиотека понадобится
> в Сизифе.

В таком случае предлагаю назвать старую libvalhalla, если она кому-нибудь понадобится как libvalhalla-media или подобным образом.

Рецензента нет уже месяц. Прошу призвать его.

Рецензента нет уже длительное время. Прошу призвать его.

На этой неделе рецензента не будет уже два месяца. Прошу призвать его.

Рецензента нет уже три месяца. Очень прошу призвать его.

Рецензента нет уже четыре месяца. Очень прошу призвать его.

Рецензента нет уже пять месяцев. Очень прошу призвать его.

Решил сам посмотреть.

== Общее замечание ==
Самая важная часть работы над пакетами это взаимодействие с другими
разработчиками (самый важный и поучительный пример «другого
разработчика» это ты сам, но спустя несколько лет, когда подробности и
причины уже вымылись из памяти).  Это значит, что commit messages, rpm
changelog и комментарии в коде это крайне важные инструменты, которыми
важно пользоваться; они нужны не только для того, чтобы инструменты были
довольны.

== По всем пакетам ==
* Ни в одном git-репозитории нет бранчей, во-первых должны быть бранчи,
  во-вторых должен быть default-branch который указывает (см. ssh
  git.alt default-branch --help).  Без этого репозиторий практически
  непригоден для взаимодействия (в том числе для рецензирования);
  выглядит это так, будто автор считает опубликованный git-репозиторий
  не более чем неизбежным этапом для запихивания пакета в сборочницу, а
  не инструмент для совместной разработки.
* "Change architecture list" это абсолютно неясное описание изменение, я
  бы написал в коммите что-то вроде "spec: use %java_arches macro to
  build only on supported architectures" и в rpm changelog что-то вроде
  "Changed to use %%java_arches macro instead of building as noarch".
* Не требующее исправления ("мнение"): подход с использованием каталога
  .gear для spec-файла, патчей и прочих дополнительных файлов я считаю
  неправильным. Этот подход достаточно распространён, поэтому сложно
  засунуть этого джина обратно в бутылку, когда половина команды так
  делает, но обратить внимание всё равно хочется. .gear это каталог для
  служебных файлов, используемых gear, туда *можно* положить файлы, но
  это нецелевое его использование. Для spec-а и прочих дополнительных
  файлов я обычно использую каталог alt, но он может называться как
  угодно.
* Не требующее исправления ("полезно знать"): если у подпакета нет секции %files, наличие
  %package <subpackage> не имеет никакого эффекта, т.о. изменение такого типа
	+%if_with java
	 %package ...
  имеет лишь косметический эффект.

== apache-rat ==

== Про новые java-пакеты ==
* Это явная ошибка:
	BuildRequires: /proc
  возможно, не в вашем пакете, а в самой java, но тем не менее (см.
  4.2.1 в /usr/share/doc/hasher-*/FAQ);
* Выглядит будто apicatalog-titanium-rdf и apicatalog-titanium-rdfc это
  одно и то же.

== beust-jcommander ==
* Вот очень неправильный подход:
	+%if_with check
	+Patch3: 0004-Port-to-Java-11-for-tests-alt-patch.patch
	+%endif
  патчи должны прикладываться вне зависимости от того, включены ли тесты
  или на какой архитектуре собирается пакет, потому что иначе плодятся
  сущности и получаются две разные версии кода в одном и том же пакете.

== biz-aQute-bnd-gradle-plugins, gradle ==
* "- Noarch packaging.", лучше было бы написать "Built as a noarch
  package".

== checker-framework ==
* Вместо
	- Add checker-dataflow subpackage.
	- Add checker-dataflow-errorprone subpackage.
	- Add checker-dataflow-nullaway subpackage.
	- Add checker-dataflow-shaded subpackage.
	- Add checker-javacutil subpackage.
	- Add checker-util subpackage.
  можно было написать:
  	- Added subpackages:
	  + checker-dataflow;
	  + checker-dataflow-errorprone;
	  ...

== foreman ==
* В сообщении "- Introduce ExclusiveArch for Java." совершенно напрасно
  не упомянут %%java_arches (или даже прямо "%%java_arches macro from
  rpm-macros-java package").

== gradle ==
* Тут явное противоречие:
	+%ifnarch %ix86
	...
	BuildArch: noarch
  
  "BuildArch: noarch" и %ifarch/%ifnarch могли бы иметь смысл вместе
  только если бы было указано для подпакета, а тут получается условие,
  которое будет всегда верно: если архитектура noarch, то она никогда не
  совпадёт с %ix86.

Продолжение будет после того, как будет исправлена проблема с самими
git-репозиториями.
В целом, всё выглядит достаточно неплохо, но пока что есть ощущение, что
пакеты и изменения все достаточно однотипны.