Bug 44756

Created attachment 12176 [details]
gpg key

Подтверждаю участие в качестве ментора.

Created attachment 12178 [details]
gpg key

В качестве вступительного задания создал пакет для lazygit: https://github.com/Turbid/alt.lazygit

Кандидат готов перейти на этап 2.

(In reply to Илья Демьянов from comment #0)
> Created attachment 12175 [details]
> ssh key
Ok.
(In reply to Илья Демьянов from comment #3)
> Created attachment 12178 [details]
> gpg key
Ok.

(Ответ для Slava Aseev на комментарий #5)
> Кандидат готов перейти на этап 2.

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

T/J/S -> 2.3.

Получил доступ к git.alt.

В качестве теста обновил один пакет sngrep из апстрима: https://git.altlinux.org/people/turbid/packages/?p=sngrep.git;a=summary

Comment on attachment 12176 [details]
gpg key

-----BEGIN PGP PUBLIC KEY BLOCK-----

mQINBGPadZwBEADXfxAAl+dzkDSEcA8fQnfbbrjwpppmoUcALml4+kOU7+tptMTG
f7lgAMvyC4z/gZPEFEmj70ihMewod+jFwktvuqbRqoWhcgDABlYk5mXC6V0r0wxp
W2k2BKqR5bxbyxl6jR45gR6IFKjo0aEkr7VC1ZzFB0gP6PadaW8VePuw7stQD+cf
5umvD3gpiuZljjuQoy/FZBFUTdi8NLHNoJOfb9PT9RHSq7ZYPREKFqh8AbkauGWI
I0Lc+26D9UxLGLBKSR9CPNwGRPE14guK/jn7YzsayiWaWBaiAUE8HkhMJcVtbrHC
z+p996pTKCqc/zyzpwr0yyxeApbqWnbVI1WD6en5ZXF/2/4THh7Kxf/NnbT9rOTh
2G8qeKQdCK6fSGfPPtwGkQyLTbZ7xK75KizdUKK7NJx6Yiye4eXbmfff+rEhmKa8
H5jH2ogPAJgTvt4pHvEwjUsfZs8STG1VRLjF4WhZbWGkXXwid8w8rjzdAhSTCmej
6+fk5sIbgkf3HPHlAMQ2B+DCz1HFzfoN2PSxiQzbsLwHgxnwcytp38+cUOKaBdK7
HjVVNdkTNsYy/kdKsda6hdB9egSmaqAULJFzD0U18OwQaIj7xxcxCjnEJ0PClM+3
1yVu0tCdfOsDIn8BQkfRim5EUowadudGnTAB1g9oczTsltlOIu50u6GWtQARAQAB
tCNJbHlhIERlbXlhbm92IDx0dXJiaWRAYWx0bGludXgub3JnPokCTgQTAQgAOBYh
BKDR9G8B9vwhZWP5wzyhlkPOIRi1BQJj2nWcAhsDBQsJCAcCBhUKCQgLAgQWAgMB
Ah4BAheAAAoJEDyhlkPOIRi1CjsQAJlZ1VvloV44mxy7dotQf5+7lfodmiVu1OCc
MlUdC9SzKvCNLEJmRXwMoSMyXsrHniqf5NOqWeIGbLArMzAf8Bwp8tD2xLZhXmn7
i1BS/jS/tjc2OFcs9K8xo69qhQ1fSugmqqW3dqkCYUAVUYF8hCkNULGV/3riQHJ+
UQ5B4Y3n+xz/FzPpp7wmpAVazdESRb/raqXKoXmpIaqFgGD3/zYuOSNLQliWhms1
mne8e+86bpuCaN+Oimllijba70tM5HdpMcc9vuLgofQbDViUpbDVXy1MIfoEJqzY
llzH+El2E95gPe59AOJuGAFvbfsWe7NTIsvaM8Pfnpw6lIFUw0C6mdUq8Net//yk
rbtYyTG7Jy4vHkDuo6XnHgYGhUfRABTWY9h7cdBzZCW+bBrWxWKB7s2YAKlWfa/L
nyXUO9f0NtoRn7gTart1POCG67YrkgCCadsNJIGboTPSZ0SWTZwua7DL2SjPPB1b
Wmi70mb/FT7QCtt8dW7YkJcIYxS5TEWgaD4BW+5wNpsCrJUvAeH8XHebCTImitHI
F5L5wHqIPyN8DhWrbOEd2syedhn4JWzOnS8gAP3Gg/Yde0kWc3p4trXVZqXPM5s4
OZu3XOUyFxqrRBuClNQ86tc6C1stEsWh9fP+nmjSUfDXpY2cUhaWmhWNWAWMGBkM
OkqQNLf+uQINBGPadZwBEADZt0RKgmnMuKOqPgtzY3aliCILfE5CF7p4lVwugkr/
ajbn+/y4e2Zu9zjbN25hmx4ED2tr20GBr2wn18LYLnC6kMQnbaP1cExfAJestbfz
7AWhsFa9inaWF6sEX4DKxM9eMmAjZMUHRudsmSVwRj/XZGG9ANCfOdCOS4IXHmQG
peDYBzqc7m4w3uOKVCBLgxkNBL6ITYLjVYe5LrU1XU4hanIru73nszlRRC4gSXvf
xRvy5B7tGRORM+CmBGJ+FsxXW4YsGjqyTTG+F85uzE4rgezmVeWSySfugpPZ7qUN
xlhyCBNl8jZv1siFdjZOIFuzpJNI4z707ni326ONq+l0uROWcXaWzzt1mqN8mzxk
H+xe8AkHA4UlttZJxjy069UrRra+Q7uVDwhEop9L9Sh3mM3dBPP0cL7ErH2feY1G
XFfsmYF5O4vR/XiXN1vyseAojXdN2s6fv8EvfTG9d4QRqoyV1SL2DpUQtJRmBkdj
I5Xtkpqv3YUsk3zoOENehjF1qGPikPlPIq7W9WAMHgqk5aZ1NDcQTpDJRKa4VGpW
8TxL5UeTmg59tMJYixFDzrBSfAYOEwxxcEvnq9Zgi7aHSfF/Apl0vyCyCCHgVgDd
94yaugtWoocbYhNywFZSjy0mSThdCtZ1P56ZulIHK8l/HqSc3cRB5F6fKr2h8N7K
WwARAQABiQI2BBgBCAAgFiEEoNH0bwH2/CFlY/nDPKGWQ84hGLUFAmPadZwCGwwA
CgkQPKGWQ84hGLXhTxAAzANpeRjQYOGtPwGjXLIt6XfC5ZgPAbJWbYuxbPyC2ItM
1ctZ3iaj/+4bL3k8PDMeNAWBKjmcjSY8RHC4y6yBSpb6TtzNJfuaQ3mxxdPMIfJA
tjfQ0trpsHjlGF5A8S7sShwKKSCcb3I4GRtT9mm94Pyw0Vt9C6+zXWWeaWQZRIz/
9qwbtM68qvSuJW+gEoIwDi6k+q7N5DwIfFsSnZL3ye4bWsFh8VIFXx2mTE77oxf/
JPgRuLOKriDkSadRb0OXwVs1c2v2cNjVvg/clmbjkj8hq3Vt0HXwPOHAR8CQFaH0
5zEUUsH3gJZmNyHw64Si2CwN1TMAgrOxXqv6L4Nq0e97JYTqros+/0Z99UDwuJ6w
vhNz2s0qED9qUGeH9y8n+yaTBqaKD+zy+d9nZumT4Moxsy8E2NyZJX1QwOjB0XDH
Rd85OCCm+46gU+HRGcfmSBKr+mDegQlkVPZ/B4MN3KZZ4iTp7idIgUvs+9pdfClA
HXnxBcgWXFkzssuJsDHuMwt3C5/1bmz30tq5kr6qVex36chBTvKb03gkRQawb6uK
p9wvDgNfsIj7dIfNY9EZS4UiqF7+0cHG3l7XmF5ZE3MfYf/0fEx2ceIht7s9KZAQ
nm546zj4zDcEY8zKMhCNX04Eew2teUjcp/sFjtDcxjqF9vKbK9E7/9DSwzYwCXc=
=Zjz+
-----END PGP PUBLIC KEY BLOCK-----

Created attachment 12391 [details]
gpg key

актуальный gpg

Кандидат готов перейти на этап 3, прошу дать доступ на сборочницу.

Также, обратите внимание, что кандидат пожелал использовать другой gpg ключ (но раз он еще не был никуда добавлен, то, полагаю, особых проблем с этим быть не должно).

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

T/J/S -> 3.5.

Приветствую.

Выполнил тестовые сборки:
- Новый пакет lazygit: https://packages.altlinux.org/ru/tasks/315885/
- Обновленные пакеты, которые выглядили заброшенными:
  - xkb-switch: https://packages.altlinux.org/ru/tasks/315882/
  - sngrep: https://packages.altlinux.org/ru/tasks/315888/
  - lf: https://packages.altlinux.org/ru/tasks/315889/

обновил пакеты:
#319765 TESTED #1 [test-only] sisyphus lazygit.git=0.38.1-alt1
#319758 TESTED #1 [test-only] sisyphus lf.git=r29-alt1
#318054 TESTED #3 [test-only] sisyphus sngrep.git=1.7.0-alt1

Считаю, что кандидат готов перейти на следующий этап.
Прошу призвать рецензента для независимой оценки готовности вступления в Team.

Обновил пакеты:
>#321914 TESTED #1 [test-only] sisyphus lazygit.git=0.38.2-alt1
>#321911 TESTED #1 [test-only] sisyphus lf.git=r30-alt1

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

T/J/S -> 4.2.

1 lazygit
1.1 Пожалуйста, делайте всегда импорт исходников отдельным коммитом. Прошу так больше не делать.
1.2 В спеках Альта принято не заключать макросы в фигурные скобки, если это не требуется для отделения макроса от окружающего его текста.
1.3 Паковать
%doc LICENSE
излишне. Все лицензии в /usr/share/docs/
Лицензия пакета указана в спеке и этого достаточно.
1.4 Рекомендация. Если есть пакет с макросами, то в
BuildRequires(pre):
нужно указывать его, а rpm-build- пакет в BuildRequires:
В данном случае вместо
BuildRequires(pre): rpm-build-golang
лучше сделать
BuildRequires(pre): rpm-macros-golang
BuildRequires: rpm-build-golang
Т.е. в BuildRequires(pre) указываются только пакеты с макросами.
1.5 Теперь по .gear/rules
Сейчас там просто:
tar: .
Так как исходники получаете из архива, то гораздо удобнее импортировать их в отдельный каталог и потом этот каталог паковать в исходники.

Жаль, что ментор пропустил пакет в Сизиф в таком виде.

2. К lf вопросов нет. Пакет обновили, каких-то своих изменений не вносили.

3. К sngrep вопросов нет. Обновили нормально, мигрировали на pcre2.

4. xkb-switch
4.1 Идентификатор лицензии у пакета не GPL-3.0, а GPL-3.0-or-later
В тексте есть такой абзац:
https://git.altlinux.org/tasks/archive/done/_308/315882/gears/100/git?p=git;a=blob;f=COPYING;h=94a9ed024d3859793618152ea559a168bbcbb5e2;hb=31f1074594d3857b6e48c1437291f56569eaf014#l637
4.2 У пакетов без мантейнеров стоит удалять поле Packager
4.3 BuildPreReq устарел, заменяйте его на BuildRequires
4.4 Пакет не перевели на использование макросов cmake. Переведите, пожалуйста
4.5 Опакечен симлинк /usr/lib64/libxkbswitch.so
Симлинк без soname опакечиваться должен только в devel пакеты. Но так как тут devel пакет не предусмотрен, предлагаю отключить сборку библиотек опцией cmake.

Предлагаю поправить lazygit и xkb-switch или оспорить мои замечания.

(Ответ для Антон Мидюков на комментарий #19)
> 4.3 BuildPreReq устарел, заменяйте его на BuildRequires

Тут я не прав. BuildPreReq нужно использовать при автоматической генерации зависимостей при помощи buildreq [1]. Это синоним BuildRequires, но при очередном выполнении buildreq не трогает строчки с BuildPreReq. Т.е. туда помещаем те зависимости, которые не находит buildreq.
В данном конкретном спеке замена BuildPreReq на BuildRequires вполне правильное решение.
Напомню, что rpm-macros-cmake должен быть в BuildRequires(pre) (сейчас всё в кучу свалено там).
 
1. https://www.altlinux.org/Buildreq

P.s.: спасибо Глебу, что напомнил мне про buildreq.

>1.1 Пожалуйста, делайте всегда импорт исходников отдельным коммитом. Прошу так больше не делать.
>...
>Жаль, что ментор пропустил пакет в Сизиф в таком виде.

Есть ли возможность (и необходимость) начать с нуля или глобально переделать организацию пакета? Хотелось бы вести пакет с сохранением коммитов апстрима и задействовать механизм gears/remote.

>Симлинк без soname опакечиваться должен только в devel пакеты

Без симлинка не работает плагин для neovim: https://github.com/ivanesmantovich/xkbswitch.nvim/blob/master/lua/xkbswitch.lua#L26

Остальные замечания понятны, буду исправлять

(Ответ для Илья Демьянов на комментарий #21)
> >1.1 Пожалуйста, делайте всегда импорт исходников отдельным коммитом. Прошу так больше не делать.
> >...
> >Жаль, что ментор пропустил пакет в Сизиф в таком виде.
> 
> Есть ли возможность (и необходимость) начать с нуля или глобально переделать
> организацию пакета? Хотелось бы вести пакет с сохранением коммитов апстрима
> и задействовать механизм gears/remote.
> 

Думаю, что необходимости такой нет. Предлагаю перенести исходники в отдельный каталог. Мне нравится обновлять командой
rpmgs -f <версия>

Это тоже удобно.

Я считаю, что смысл собирать пакет из апстримного git есть тогда, когда пакет со стороны мантейнера часто патчится и даже иногда апстримится. Удобно делать свои коммиты в git, а патч генерировать уже при сборке. В таких случаях в .gear/rules прописывают:
tar: v@vesion@:.
diff: v@vesion@:. .

В спеке прописывают:
Patch: %name-%version-%release.patch
Если есть планы патчить пакет и апстримить свои изменения, то можно перейти на схему мержа с апстримным git. Просто сделайте мерж с апстримным тегом и поправьте .gear/rules и спек.

> >Симлинк без soname опакечиваться должен только в devel пакеты
> 
> Без симлинка не работает плагин для neovim:
> https://github.com/ivanesmantovich/xkbswitch.nvim/blob/master/lua/xkbswitch.
> lua#L26

Так ему нужен libxkbswitch.so.1, а не libxkbswitch.so
Не опакечивайте libxkbswitch.so

(Ответ для Антон Мидюков на комментарий #19)
> 1.3 Паковать
> %doc LICENSE
> излишне. Все лицензии в /usr/share/docs/
> Лицензия пакета указана в спеке и этого достаточно.

В тексте MIT сказано буквально следующее:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

Указание лицензии MIT в спеке не является копией самого текста "permission notice".

Поэтому предлагаю всё-таки паковать текст лицензии.

(Ответ для Grigory Ustinov на комментарий #23)
> (Ответ для Антон Мидюков на комментарий #19)
> > 1.3 Паковать
> > %doc LICENSE
> > излишне. Все лицензии в /usr/share/docs/
> > Лицензия пакета указана в спеке и этого достаточно.
> 
> В тексте MIT сказано буквально следующее:
> 
> The above copyright notice and this permission notice shall be included in
> all copies or substantial portions of the Software.
> 
> Указание лицензии MIT в спеке не является копией самого текста "permission
> notice".
> 
> Поэтому предлагаю всё-таки паковать текст лицензии.

Согласен. Спасибо.

В тексте лицензии успользована формулировка "shall be included", что открывает возможность ещё одного варианта: запаковать ссылку на файл с текстом лицензии, тогда название лицензии будет видно, даже не открывая файл.  Например,

ln -s %_licensedir/MIT LICENSE
%doc --no-dereference LICENSE

Прошу проверить исправление замечаний:

>#327334 TESTED #1 [test-only] sisyphus xkb-switch.git=1.8.5-alt2
>#327326 TESTED #1 [test-only] sisyphus lazygit.git=0.40.2-alt1

(Ответ для Илья Демьянов на комментарий #26)
> Прошу проверить исправление замечаний:
> 
> >#327334 TESTED #1 [test-only] sisyphus xkb-switch.git=1.8.5-alt2

Для того, чтобы макросы отображались в changelog текстом их нужно экранировать ещё одним знаком % перед макросом:
- Change "quick fix" CMakeLists.txt from sed to %%__subst macro and move to %prep

Сейчас строка выглядит так:
- Change "quick fix" CMakeLists.txt from sed to /usr/bin/subst macro and move to %prep

Также эту строку следует разбить на две, так как она больше 80 символов. Делается с таким отступом:
- Первая строка до 80 символов
  продолжение на второй строке

> >#327326 TESTED #1 [test-only] sisyphus lazygit.git=0.40.2-alt1

нормально.

>Для того, чтобы макросы отображались в changelog текстом их нужно экранировать ещё одним знаком % перед макросом

Ох... Как лучше это исправить? Создать еще одину версию alt3 или reset последнего коммита?

(Ответ для Илья Демьянов на комментарий #28)
> >Для того, чтобы макросы отображались в changelog текстом их нужно экранировать ещё одним знаком % перед макросом
> 
> Ох... Как лучше это исправить? Создать еще одину версию alt3 или reset
> последнего коммита?

Переделать коммит.

>Переделать коммит.

Готово, в рамкам того же task

(Ответ для Илья Демьянов на комментарий #30)
> >Переделать коммит.
> 
> Готово, в рамкам того же task

Хорошо. Одобрил оба задания.
Ничего больше не собирали?

>Ничего больше не собирали?

Пока нет, пытаюсь разбираться с Asterisk: https://bugzilla.altlinux.org/46017

(Ответ для Илья Демьянов на комментарий #32)
> >Ничего больше не собирали?
> 
> Пока нет, пытаюсь разбираться с Asterisk: https://bugzilla.altlinux.org/46017

Хорошо. Думаю, что кандидат теперь готов.

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

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