Created attachment 12175 [details] ssh key Добрый день. Имя: Илья Демьянов Псевдоним: turbid Адрес для пересылки: cavetroll@yandex.ru Ментор: Slava Aseev <ptrnine@altlinux.org> Планы: актуализация и поддержка пакетов для VoIP - прежде всего asterisk, sngrep, baresip. Спасибо.
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@. Пользователь добавлен в группу мейнтейнеров. Желаю удачного мейнтейнерства!
