Bug 44756

Summary: [done] join turbid@
Product: Team Accounts Reporter: Илья Демьянов <cavetroll>
Component: joinAssignee: Gleb F-Malinovskiy <glebfm>
Status: CLOSED FIXED QA Contact: Andrey Cherepanov <cas>
Severity: normal    
Priority: P5 CC: antohami, glebfm, grenka, ldv, ptrnine
Version: unspecified   
Hardware: x86_64   
OS: Linux   
URL: https://altlinux.org/Team/Join
Attachments:
Description Flags
ssh key
none
gpg key
none
gpg key
none
gpg key none

Description Илья Демьянов 2022-12-28 14:06:46 MSK
Created attachment 12175 [details]
ssh key

Добрый день.

Имя: Илья Демьянов
Псевдоним: turbid
Адрес для пересылки: cavetroll@yandex.ru

Ментор: Slava Aseev <ptrnine@altlinux.org>

Планы: актуализация и поддержка пакетов для VoIP - прежде всего asterisk, sngrep, baresip.

Спасибо.
Comment 1 Илья Демьянов 2022-12-28 14:08:09 MSK
Created attachment 12176 [details]
gpg key
Comment 2 Slava Aseev 2022-12-28 14:16:46 MSK
Подтверждаю участие в качестве ментора.
Comment 3 Илья Демьянов 2022-12-28 15:12:06 MSK
Created attachment 12178 [details]
gpg key
Comment 4 Илья Демьянов 2023-01-10 12:47:01 MSK
В качестве вступительного задания создал пакет для lazygit: https://github.com/Turbid/alt.lazygit
Comment 5 Slava Aseev 2023-01-10 12:53:08 MSK
Кандидат готов перейти на этап 2.
Comment 6 Gleb F-Malinovskiy 2023-01-10 12:57:48 MSK
(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.
Comment 7 Slava Aseev 2023-01-10 14:08:12 MSK
(Ответ для Slava Aseev на комментарий #5)
> Кандидат готов перейти на этап 2.
Comment 8 Gleb F-Malinovskiy 2023-01-16 21:29:18 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.
Comment 9 Илья Демьянов 2023-02-01 16:04:52 MSK
Получил доступ к git.alt.

В качестве теста обновил один пакет sngrep из апстрима: https://git.altlinux.org/people/turbid/packages/?p=sngrep.git;a=summary
Comment 10 Илья Демьянов 2023-02-01 17:58:08 MSK
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-----
Comment 11 Илья Демьянов 2023-02-01 18:02:34 MSK
Created attachment 12391 [details]
gpg key

актуальный gpg
Comment 12 Slava Aseev 2023-02-01 18:35:15 MSK
Кандидат готов перейти на этап 3, прошу дать доступ на сборочницу.

Также, обратите внимание, что кандидат пожелал использовать другой gpg ключ (но раз он еще не был никуда добавлен, то, полагаю, особых проблем с этим быть не должно).
Comment 13 Gleb F-Malinovskiy 2023-02-13 11:44:11 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.

T/J/S -> 3.5.
Comment 14 Илья Демьянов 2023-02-27 11:13:34 MSK
Приветствую.

Выполнил тестовые сборки:
- Новый пакет 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/
Comment 15 Илья Демьянов 2023-05-03 13:34:18 MSK
обновил пакеты:
#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
Comment 16 Slava Aseev 2023-05-10 13:52:17 MSK
Считаю, что кандидат готов перейти на следующий этап.
Прошу призвать рецензента для независимой оценки готовности вступления в Team.
Comment 17 Илья Демьянов 2023-05-26 12:58:53 MSK
Обновил пакеты:
>#321914 TESTED #1 [test-only] sisyphus lazygit.git=0.38.2-alt1
>#321911 TESTED #1 [test-only] sisyphus lf.git=r30-alt1
Comment 18 Gleb F-Malinovskiy 2023-06-16 13:33:49 MSK
Призван рецензент (antohami@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.
Comment 19 Антон Мидюков 2023-06-16 16:23:07 MSK
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 или оспорить мои замечания.
Comment 20 Антон Мидюков 2023-06-16 18:55:58 MSK
(Ответ для Антон Мидюков на комментарий #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.
Comment 21 Илья Демьянов 2023-06-16 23:28:24 MSK
>1.1 Пожалуйста, делайте всегда импорт исходников отдельным коммитом. Прошу так больше не делать.
>...
>Жаль, что ментор пропустил пакет в Сизиф в таком виде.

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

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

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

Остальные замечания понятны, буду исправлять
Comment 22 Антон Мидюков 2023-06-17 07:40:39 MSK
(Ответ для Илья Демьянов на комментарий #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
Comment 23 Grigory Ustinov 2023-07-11 16:10:58 MSK
(Ответ для Антон Мидюков на комментарий #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".

Поэтому предлагаю всё-таки паковать текст лицензии.
Comment 24 Антон Мидюков 2023-07-11 16:35:00 MSK
(Ответ для 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".
> 
> Поэтому предлагаю всё-таки паковать текст лицензии.

Согласен. Спасибо.
Comment 25 Dmitry V. Levin 2023-07-11 22:02:48 MSK
В тексте лицензии успользована формулировка "shall be included", что открывает возможность ещё одного варианта: запаковать ссылку на файл с текстом лицензии, тогда название лицензии будет видно, даже не открывая файл.  Например,

ln -s %_licensedir/MIT LICENSE
%doc --no-dereference LICENSE
Comment 26 Илья Демьянов 2023-08-17 18:11:15 MSK
Прошу проверить исправление замечаний:

>#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
Comment 27 Антон Мидюков 2023-08-17 20:06:30 MSK
(Ответ для Илья Демьянов на комментарий #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

нормально.
Comment 28 Илья Демьянов 2023-08-17 21:22:14 MSK
>Для того, чтобы макросы отображались в changelog текстом их нужно экранировать ещё одним знаком % перед макросом

Ох... Как лучше это исправить? Создать еще одину версию alt3 или reset последнего коммита?
Comment 29 Антон Мидюков 2023-08-18 04:42:13 MSK
(Ответ для Илья Демьянов на комментарий #28)
> >Для того, чтобы макросы отображались в changelog текстом их нужно экранировать ещё одним знаком % перед макросом
> 
> Ох... Как лучше это исправить? Создать еще одину версию alt3 или reset
> последнего коммита?

Переделать коммит.
Comment 30 Илья Демьянов 2023-08-18 09:21:54 MSK
>Переделать коммит.

Готово, в рамкам того же task
Comment 31 Антон Мидюков 2023-08-18 09:29:27 MSK
(Ответ для Илья Демьянов на комментарий #30)
> >Переделать коммит.
> 
> Готово, в рамкам того же task

Хорошо. Одобрил оба задания.
Ничего больше не собирали?
Comment 32 Илья Демьянов 2023-08-18 10:00:59 MSK
>Ничего больше не собирали?

Пока нет, пытаюсь разбираться с Asterisk: https://bugzilla.altlinux.org/46017
Comment 33 Антон Мидюков 2023-08-18 10:16:09 MSK
(Ответ для Илья Демьянов на комментарий #32)
> >Ничего больше не собирали?
> 
> Пока нет, пытаюсь разбираться с Asterisk: https://bugzilla.altlinux.org/46017

Хорошо. Думаю, что кандидат теперь готов.
Comment 34 Gleb F-Malinovskiy 2023-08-30 13:02:40 MSK
Адрес подписан на devel@.
Пользователь добавлен в группу мейнтейнеров.

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