Bug 45885 - [done] join skachedubov@
Summary: [done] join skachedubov@
Status: CLOSED FIXED
Alias: None
Product: Team Accounts
Classification: Development
Component: join (show other bugs)
Version: unspecified
Hardware: all Linux
: P5 normal
Assignee: Gleb F-Malinovskiy
QA Contact: Andrey Cherepanov
URL: https://altlinux.org/Team/Join
Keywords:
Depends on:
Blocks:
 
Reported: 2023-04-17 14:06 MSK by Danila Skachedubov
Modified: 2026-02-05 13:35 MSK (History)
6 users (show)

See Also:


Attachments
Прикрепляю публичную часть SSH ключа (101 bytes, application/vnd.ms-publisher)
2023-05-02 14:46 MSK, Danila Skachedubov
no flags Details
Прикрепляю публичную часть GPG ключа (3.03 KB, application/vnd.apple.keynote)
2023-05-02 14:48 MSK, Danila Skachedubov
no flags Details
Прикрепляю публичную часть SSH ключа (106 bytes, application/vnd.ms-publisher)
2023-05-02 15:57 MSK, Danila Skachedubov
no flags Details
GPG ключ (3.01 KB, application/vnd.apple.keynote)
2025-02-11 13:54 MSK, Danila Skachedubov
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Danila Skachedubov 2023-04-17 14:06:52 MSK
Псевдоним       : skachedubov
Почта           : Danila Skachedubov <skachedubov@altlinux.org>
Пересылка почты : SKADA2012@yandex.ru
Имя ментора     : Иван Савин
Почта ментора   : svn17@altlinux.org
Моя цель        : Научиться собирать пакеты
Comment 1 Gleb F-Malinovskiy 2023-05-02 13:35:28 MSK
Эта заявка недооформлена.
Можете переоткрыть баг когда решите её оформить.
Comment 2 Danila Skachedubov 2023-05-02 14:46:37 MSK
Created attachment 13073 [details]
Прикрепляю публичную часть SSH ключа
Comment 3 Danila Skachedubov 2023-05-02 14:48:19 MSK
Created attachment 13074 [details]
Прикрепляю публичную часть GPG ключа
Comment 4 Danila Skachedubov 2023-05-02 15:57:40 MSK
Created attachment 13078 [details]
Прикрепляю публичную часть SSH ключа
Comment 5 Иван Савин 2023-05-03 12:41:13 MSK
Подтверждаю заявку.
Прошу секретаря зарегистрировать ключи.
Comment 6 Иван Савин 2023-05-10 12:17:21 MSK
Считаю что кандидат уже умеет генерировать ключи и готов к переходу на следующий этап.
Comment 7 Gleb F-Malinovskiy 2023-05-26 11:46:15 MSK
(In reply to Danila from comment #4)
> Created attachment 13078 [details]
> Прикрепляю публичную часть SSH ключа
Ok.

(In reply to Danila from comment #3)
> Created attachment 13074 [details]
> Прикрепляю публичную часть GPG ключа
Я не понимаю, зачем нужен такой комментарий к ключу, но формально ключ, конечно, соответствует критериям.
Comment 8 Иван Савин 2023-05-29 12:37:54 MSK
Кандидат готов начать встаупление в team.
Comment 9 Gleb F-Malinovskiy 2023-06-08 18:17:01 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.     

T/J/S -> 2.3.
Comment 10 Иван Савин 2023-06-09 12:02:34 MSK
Прошу кандидата предоставить примеры пакетов на git.altlinux.org.
Comment 11 Danila Skachedubov 2023-12-07 13:36:38 MSK
Добавил несколько пакетов: https://git.altlinux.org/people/skachedubov/packages/
1. extarct-dtb: инструмент для разделения образа ядра с добавленными dtb на отдельные файлы ядра и dtb.
2. StegoPy: инструмент для стеанографии методом LSB.
3. spi-tools: инструмент командной строки, который помогает использовать устройства Linux spidev.
4. vunit: - фреймворк для Unit-тестов для Verilog.
Comment 12 Иван Савин 2023-12-22 14:54:34 MSK
(Ответ для Danila Skachedubov на комментарий #11)
> Добавил несколько пакетов:
> https://git.altlinux.org/people/skachedubov/packages/
> 1. extarct-dtb: инструмент для разделения образа ядра с добавленными dtb на
> отдельные файлы ядра и dtb.
> 2. StegoPy: инструмент для стеанографии методом LSB.
> 3. spi-tools: инструмент командной строки, который помогает использовать
> устройства Linux spidev.
> 4. vunit: - фреймворк для Unit-тестов для Verilog.

Посмотрел первый пакет из списка.
Тег Packager надо убрать, он больше не используется.
Тег License сделать в соответствии с https://www.altlinux.org/Spec#License.

Исправить:
warning: Installed (but unpackaged) file(s) found:
    /usr/CHANGES.md
    /usr/LICENSE
    /usr/bin/extract-dtb
    /usr/lib/python3/site-packages/extract_dtb-1.2.4.dev0.dist-info/METADATA
    /usr/lib/python3/site-packages/extract_dtb-1.2.4.dev0.dist-info/entry_points.txt

Остальные пакеты пока не смотрел, сказанное выше актуально и для них.
Comment 13 Gleb F-Malinovskiy 2024-03-27 18:26:25 MSK
Ещё актуально?
Comment 14 Danila Skachedubov 2024-03-28 16:51:16 MSK
Внес изменения в пакеты:
https://git.altlinux.org/people/skachedubov/packages/
1. Удалил тэг Packager
2. Лицензии указал макросами
3. Исправил warning'и при сборке пакета в хэшере.
Comment 15 Иван Савин 2024-05-14 13:28:24 MSK
(Ответ для Danila Skachedubov на комментарий #14)
> Внес изменения в пакеты:
> https://git.altlinux.org/people/skachedubov/packages/
> 1. Удалил тэг Packager
> 2. Лицензии указал макросами
> 3. Исправил warning'и при сборке пакета в хэшере.

Посмотрел второй пакет из списка. Он не собирается.
Comment 16 Danila Skachedubov 2024-05-15 14:49:49 MSK
Исправил ошибки в spec файлах:
1. Изменил домен в почте на правильный (altlinux.org).
2. Исправил путь в секции %install в пакете StegoPy
Comment 17 Иван Савин 2024-07-03 12:19:56 MSK
Первый пакет:
1) Смешивать свои коммиты с коммитами апстрима не очень хорошая идея по моему.
   Наверное, лучше свои коммиты делать в отдельной ветке.

Второй пакет:
1) первый пункт для первого пакета.
   https://www.altlinux.org/Spec#Version - про версии.
2) Почему не использовать %_bindir вместо %define install_dir /usr/bin/ ?

Третий пакет:
1) первый пункт для первого пакета.
2) зачем эта переменная - %define version 4.7.0?

Четвёртый пакет:
1) Первый пункт из первых трёх можно посмотреть здесь.
2) Ветку по умолчанию логичней назвать sisyphus и собирать из неё, зачем master?
Comment 18 Danila Skachedubov 2024-08-20 14:13:12 MSK
Исправил ошибки:
Разделил ветки с апстримом и своими коммитами со сборкой
Исправил использование макросов в spec файлах
Разобрался с промежуточными версиями для пакета StegoPy
Comment 19 Danila Skachedubov 2024-10-14 14:24:43 MSK
 - Исправил ошибку при указании промежуточной версии, созданной командой git describe
 - Установил дефолтный бранч sisyphus во всех репозиториях командой ssh gitery default-branch
Comment 20 Иван Савин 2024-11-01 17:55:19 MSK
extract-dtb:
1)https://www.altlinux.org/Spec#%25description
"Длина каждой строки не должна превышать 72 символа"


StegoPy:
1)Пакет не собирается.

2) v0.0.4beta-14-g823a92b - странная версия.
https://www.altlinux.org/Spec#Version

3)https://www.altlinux.org/Spec#%25description
"Длина каждой строки не должна превышать 72 символа"

4) из https://bugzilla.altlinux.org/show_bug.cgi?id=45885#c17
> 2) Почему не использовать %_bindir вместо %define install_dir /usr/bin/ ?

spi-tools:
1)Пакет не собирается.

vunit:
1) из https://bugzilla.altlinux.org/show_bug.cgi?id=45885#c12
> Тег Packager надо убрать, он больше не используется.

2)https://www.altlinux.org/Spec#%25description
"Длина каждой строки не должна превышать 72 символа"
Comment 21 Danila Skachedubov 2024-11-02 13:58:36 MSK
Собирал пакеты командой gear-hsh --commit, исправив все недочеты и ошибки,  выявленные в предыдущей проверке, отписался в багзиллу, подсознательно считая, что закоммитил и запушил все изменения...
Прошу прощение за потраченное время на проверку(
Сейчас закоммитил и отправил все изменения, так же исправил длину строк в секции  %description.
Comment 22 Иван Савин 2024-11-29 19:15:43 MSK
StegoPy:
1) Странная версия - v0.0.4beta.14.g823a92b.
https://www.altlinux.org/Spec#Version
В примере показано как из этого делают версию.
2) Думаю стоит запушить тэг апстрима с помощью которого это получилось.

tar: . name=@name@-@version@ base=@name@-@version@ - это не ошибка, но base в данном случае дублирует действие по умолчанию (gear-rules(5)).
Comment 23 Danila Skachedubov 2024-12-02 14:30:08 MSK
Запушил тег, для получения командой git describe --abbrev=1 HEAD^ количества коммитов до последнего коммита апстрима и для формирования уникального имени объекта.
Исправил версию. 
Да, без указания base, используется имя архива без суффикса, в моем случае это излишне.
Comment 24 Иван Савин 2025-02-10 13:35:59 MSK
(Ответ для Danila Skachedubov на комментарий #23)
> Запушил тег, для получения командой git describe --abbrev=1 HEAD^ количества
> коммитов до последнего коммита апстрима и для формирования уникального имени
> объекта.
> Исправил версию. 

В версии 0.0.4.0.14.g823a один лишний символ по сравнению с примером из https://www.altlinux.org/Spec#Version
Comment 25 Danila Skachedubov 2025-02-10 17:42:36 MSK
Исправил версию 0.0.4.0.14.g823a. Удалил префикс "g".
Comment 26 Иван Савин 2025-02-10 18:12:39 MSK
Считаю, что кандидат готов к следующему шагу.
Прошу секретаря дать доступ к сборочнице.
Comment 27 Gleb F-Malinovskiy 2025-02-10 19:41:50 MSK
За прошедшее время мы ужесточили правила про gpg-ключи и принимаем ключи только без комментария.  Можно было бы рассмотреть вопрос об исключении, но я думаю, что комментарий (developer) не имеет никакого смысла.
Comment 28 Иван Савин 2025-02-11 12:02:42 MSK
Кандидат, прошу обновить ключ.
Comment 29 Danila Skachedubov 2025-02-11 13:54:34 MSK
Created attachment 17719 [details]
GPG ключ

Добрый день, прикладываю новый gpg-ключ.
Comment 30 Gleb F-Malinovskiy 2025-02-18 18:19:18 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.
Адрес подписан на devel@.

T/J/S -> 3.6.
Comment 31 Danila Skachedubov 2025-04-22 14:43:04 MSK
Добрый день! В рамках освоения работы со сборочницей собрал несколько тасок:
#381892 - таска в сизиф с пакетами, собранными для джоина.
#381890 #381882 #379482 #377866 #381957 - таски, созданные в разные ветки репозитория, собранные с флагом --test-only для тестирования вносимого функицонала в рамках решения рабочих задач.
Comment 32 Danila Skachedubov 2025-04-22 18:49:38 MSK
Добрый вечер! Пересобрал пакет cherrytree с новой версией.
https://git.altlinux.org/people/skachedubov/packages/?p=cherrytree.git;a=summary
Добавил патч, возвращающий на место флаг России, вместо установленного апстримом флага организации, запрещенной в Российской Федерации.
Данный патч закрывает багу https://bugzilla.altlinux.org/48695.
Собрал таску в сизиф 382341.
Comment 33 Иван Савин 2025-05-21 12:20:51 MSK
StegoPy, python3-module-extract-dtb, python3-module-vunit, spi-tools:

tagger Danila Skachedubov <skachedubovda@altlinux.org>

Коммитер: Danila Skachedubov <skachedubovda@altlinux.org>  2025-04-17 19:26:42

cherrytree:

tagger Danila Skachedubov <skachedubovda@altlinux.org>

Автор: Danila Skachedubov <skachedubov@altlinux.org>  2025-04-22 17:07:54
Коммитер: Danila Skachedubov <skachedubovda@altlinux.org>  2025-04-24 15:43:47

При этом в changelog'ах <skachedubov@altlinux.org> и тут
Псевдоним       : skachedubov
Почта           : Danila Skachedubov <skachedubov@altlinux.org>

В ключах тоже указан адрес skachedubov@altlinux.org.

Зачем указывать адрес <skachedubovda@altlinux.org>? Он существует?
Считаю, это надо исправить.
Comment 34 Danila Skachedubov 2025-05-21 16:21:30 MSK
Да, эта почта уже неактуальна, осталась в глобальном конфиге гита и ставила свой след в коммитах. Все исправил и запушил. Таски пересобрал, на этот раз решил не пихать пакеты не связанные друг с другом в одну таску, да и их количество не требует такого. 
#384838 TESTED #1 [test-only] sisyphus spi-tools.git=1.0.2-alt1
#384836 TESTED #1 [test-only] sisyphus StegoPy.git=0.0.4.0.14.823a-alt1
#384835 TESTED #1 [test-only] sisyphus python3-module-extract-dtb.git=1.2.3-alt1
#384833 TESTED #1 [test-only] sisyphus python3-module-vunit.git=4.7.0-alt1
#384832 EPERM #1 sisyphus cherrytree.git=1.4.0-alt1

Все tested, за исключением cherrytree, который, надеюсь с новым патчем попадет в сизиф.
Comment 35 Иван Савин 2025-06-11 13:47:18 MSK
Считаю что кандидат готов к следующему шагу. Прошу секретаря призвать рецензента.
Comment 36 Danila Skachedubov 2025-12-17 11:18:09 MSK
Добрый день, разбирался в процессе сборке/пересборке модулей ядра.
Обновил  kernel-source-rtl8192eu с новой версией, поддерживающей ядро 6.18  и модуль к нему для 6.12 и 6.18.
Уже попало в Сизиф.
https://git.altlinux.org/people/skachedubov/packages/?p=kernel-modules-rtl8192eu.git&a=shortlog&h=refs/heads/template/rtl8192eu/sisyphus

https://git.altlinux.org/people/skachedubov/packages/?p=kernel-source-rtl8192eu.git&a=shortlog&h=refs/heads/sisyphus
Comment 37 Gleb F-Malinovskiy 2025-12-23 17:40:19 MSK
Призван рецензент (antohami@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.
Comment 38 Антон Мидюков 2025-12-24 09:51:33 MSK
Замечание по уже прошедшему пакету:
https://git.altlinux.org/tasks/archive/done/_394/403606/gears/100/git?p=git;a=commitdiff;h=9041f48351e8cd0d9535b3c269b481a9b56d2832

Не делайте в одном коммите изменения в коде и спек файле.
Также вижу, что в этом спеке используются фигурные скобки в макросах: %{_datadir}
У нас принято не использовать фигурные скобки без необходимости.
____

https://git.altlinux.org/tasks/402289

тут всё нормально. Одобрил, коммитьте.
Comment 39 Антон Мидюков 2025-12-24 10:33:03 MSK
https://git.altlinux.org/tasks/401825/gears/40/git?p=git;a=commitdiff;h=2ae2202f9031a3910ddea09310b63132e257a69d

1. Укажите
VCS: https://адрес.git

2. Или выровните отступы, или уберите их:
Name:          libgeopmd
Version:       3.2.1
Release:       alt1
Summary:       C/C++ implementation of the GEOPM access service
Group: System/Libraries
License:       BSD-3-Clause
URL:           https://geopm.github.io
Source0: %name-%version.tar

3. Вместо
autoreconf -vif

нужно использовать макрос
%autoreconf

и выполнять его нужно в %build, а не %prep.

4 Нет символа новой строки в конце спека и в .gear/rules:

+- first build for ALT
\ No newline at end of file

+spec: .gear/libgeopmd.spec
\ No newline at end of file


Все текстовые файлы в linux должны заканчиваться символом новой строки.

5. Это тут ни к чему:
#BuildRequires: /sys

6. Это не нужно указывать, входят в базовый сборочный чрут:
BuildRequires: autoconf
BuildRequires: automake

7. Зачем вам пакет пустышка libgeopmd, который ничего не вытягивает? Уберите %files, чтобы его не было.

8. Почему собираете на из тега?
Comment 40 Антон Мидюков 2025-12-24 10:48:57 MSK
https://git.altlinux.org/tasks/401825/gears/400/git?p=git;a=commitdiff;h=46f64f80c95fb53b689fa342cb3d7e706d3fbcd7

1. Уж очень интересное описание у пакета:
+%global desc  test
...
+%description -n geopm-cli
+%{desc}

Неправда ли?

2. Не стоит делать так:
%_libdir/libgeopm.so.%{soversion}*
%_libdir/geopm/libgeopmiogroup_profile.so.%{soversion}*

Лучше так
%_libdir/libgeopm.so.%soversion
%_libdir/libgeopm.so.%soversion.*
%_libdir/geopm/libgeopmiogroup_profile.so.%soversion
%_libdir/geopm/libgeopmiogroup_profile.so.%soversion.*

Так гарантируется контроль %soversion
И, кстати, странно, что libgeopmiogroup_profile.so.%soversion не в %_libdir, а в подкаталоге. Обычно в подкаталоги убирают неверсионированные библиотеки, которые нужны только пакету.

3. Странная конструкция:
%if ! %defined _without_check

Можно так:
%def_with check
...
%check
%if_with_check

4. Добавьте VCS

4. Вместо
autoreconf -vif

нужно использовать макрос
%autoreconf

и выполнять его нужно в %build, а не %prep.

5. Нет символа новой строки в конце спека и в .gear/rules:

+- first build for ALT
\ No newline at end of file

+spec: .gear/libgeopmd.spec
\ No newline at end of file


Все текстовые файлы в linux должны заканчиваться символом новой строки.

6. Это тут ни к чему:
#BuildRequires: /sys

7. Это не нужно указывать, входят в базовый сборочный чрут:
BuildRequires: autoconf
BuildRequires: automake

___

Пройдитесь по своим пакетам. Скорее всего будут аналогичные замечания.
Сообщите, как всё исправите, тогда продолжу ревью.
Comment 41 Danila Skachedubov 2025-12-24 10:55:14 MSK
Добрый день, спасибо за замечания!
Хотел бы уточнить пару моментов: 
- Насколько в целом подход в пакете freeipa-server-gpo для сохранения файлов в промежуточной директории, чтобы не инсталляция в сервер происходила исключительно при запуске скрипта реализован правильно?
- Пакеты libgeopm libgeopmd python3-module-geopmpy geopmd еще не в финальном виде ,такими "сырыми" я их выложил для того, чтобы проверить работу тестов в сборочнице. Во время сборки в хешере, я столкнулся с проблемой, что для прохождения тестов нужно монтировать не только /proc, но и /sys, локально в хешере все собиралось хорошо, но при этом я ни в одном спеке не видел монтирование /sys, что и смутило меня, тесты отключать я не хотел, хотел разобраться можно ли монтировать /sys на сборочнице, что не получилось и дальше я уже экспериментировал как обходить конкретные тесты пока не добился успеха, поэтому планировал отписаться в баге как данные пакеты будут в финальном виде. Еще раз спасибо!
Comment 42 Антон Мидюков 2025-12-24 11:39:38 MSK
(Ответ для Danila Skachedubov на комментарий #41)
> - Насколько в целом подход в пакете freeipa-server-gpo для сохранения файлов
> в промежуточной директории, чтобы не инсталляция в сервер происходила
> исключительно при запуске скрипта реализован правильно?

Это действительно очень странный подход.

%post
echo "********************************************************************"
echo "FreeIPA Group Policy plugins have been installed in staging directory."
echo "To activate them, run: ipa-gpo-install"
echo "This will copy the plugins to their proper locations and configure"
echo "the necessary schema and services."
echo "********************************************************************"

Чем было вызвано?

И вот такое ужасно:
%postun
if [ $1 -eq 0 ]; then
    # Package removal (not upgrade)
    # Remove plugin files that were copied by ipa-gpo-install
    rm -f %{python3_sitelibdir}/ipaserver/plugins/chain.py
    rm -f %{python3_sitelibdir}/ipaserver/plugins/gpmaster.py
    rm -f %{python3_sitelibdir}/ipaserver/plugins/gpo.py

    rm -f %{_datadir}/ipa/ui/js/plugins/chain/chain.js
    rm -f %{_datadir}/ipa/ui/js/plugins/chain/gpo.js

    rm -f %{_datadir}/ipa/schema.d/75-chain.ldif
    rm -f %{_datadir}/ipa/schema.d/75-gpc.ldif
    rm -f %{_datadir}/ipa/schema.d/75-gpmaster.ldif

    rm -f %{_datadir}/ipa/updates/75-chain.update
    rm -f %{_datadir}/ipa/updates/75-gpc.update
    rm -f %{_datadir}/ipa/updates/75-gpmaster.update

    rm -f %{_sysconfdir}/oddjobd.conf.d/ipa-gpo.conf
    rm -f %{_prefix}/libexec/ipa/oddjob/org.freeipa.server.create-gpo-structure
    rm -f %{_prefix}/libexec/ipa/oddjob/org.freeipa.server.delete-gpo-structure
fi

Можно запаковать как %ghost всё это, чтобы так не удалять.

Я теперь посмотрел спек полностью (не только коммит), появились дополнительные вопросы:

1. Локали пакуются не так:
%{_datadir}/locale/ru/LC_MESSAGES/ipa-gpo-install.mo

Нужно в %install:
%find_lang ipa-gpo-install

А в %files прописать:
%files -f ipa-gpo-install.lang

Так они и автоматом найдутся, и получат отметку, что это локали.

2. Не нашлись автоматом:
Requires: python3-module-freeipa
Requires: python3-module-ipaserver
Requires: freeipa-server-trust-ad
Requires: samba-common-tools

?

> - Пакеты libgeopm libgeopmd python3-module-geopmpy geopmd еще не в финальном
> виде ,такими "сырыми" я их выложил для того, чтобы проверить работу тестов в
> сборочнице. Во время сборки в хешере, я столкнулся с проблемой, что для
> прохождения тестов нужно монтировать не только /proc, но и /sys, локально в
> хешере все собиралось хорошо, но при этом я ни в одном спеке не видел
> монтирование /sys, что и смутило меня, тесты отключать я не хотел, хотел
> разобраться можно ли монтировать /sys на сборочнице, что не получилось и
> дальше я уже экспериментировал как обходить конкретные тесты пока не добился
> успеха, поэтому планировал отписаться в баге как данные пакеты будут в
> финальном виде.

/sys монтировать нельзя. Видимо, тестировать придётся на altlinux.space в контейнерах, раз такие тесты. Почему туда не перемещаетесь с github, кстати?
Comment 43 Gleb F-Malinovskiy 2025-12-24 14:24:25 MSK
(In reply to Антон Мидюков from comment #42)
> /sys монтировать нельзя. Видимо, тестировать придётся на altlinux.space в
> контейнерах, раз такие тесты.
Можно же использовать vm-run, будет и /sys и настоящий root (если вдруг тоже нужно).
Comment 44 Danila Skachedubov 2025-12-29 13:58:11 MSK
Добрый день, по поводу пакета https://git.altlinux.org/people/skachedubov/packages/?p=freeipa-server-gpo.git&a=summary
Исходники есть и на altlinux.space https://altlinux.space/danila-skachedubov/freeipa-server-gpo 
В спеке поменяю ссылку с github на altlinux.space
Дабы уйти от решения в спеке в секции %postun удалять файлы и перемещать их в системе при установке плагина я пересмотрел архитектуру включения плагина, поэтому в следующей версии исправлю это. Так же учту остальные замечания.

По поводу пакетов libgeopm libgeopmd python3-module-geopmpy geopmd в которых я столкнулся с проблемой запусков тестов, я разобрался как включить kvm в хешере и запустить тесты, используя vm-run. Так же учту все замечания, указанные выше. 
Как доведу "до ума" эти пакеты, соберу заново таски и отпишусь в баге для ревью. Спасибо
Comment 45 Danila Skachedubov 2026-01-20 15:29:51 MSK
Добрый день, собрал и обновил исправленные пакеты libgeopm libgeopmd python3-module-geopmpy geopmd в таске 405230

В пакете freeipa-server-gpo пересмотрел и переделал подход к активации плагина и убрал из спека старое решение, собрал в таске 405634

Так же обновил пакет cherrytree в таске 405604

Собранные пакеты ранее по следующим таскам
python3-module-extract-dtb 405640
StegoPy 384836
python3-module-vunit 405643
spi-tools 384838
Comment 46 Антон Мидюков 2026-01-20 15:43:15 MSK
(Ответ для Danila Skachedubov на комментарий #45)
> Добрый день, собрал и обновил исправленные пакеты libgeopm libgeopmd
> python3-module-geopmpy geopmd в таске 405230

https://git.altlinux.org/tasks/405230/gears/200/git?p=git;a=commitdiff;h=fbfc60de595c706f04a58119c4ad333c53f9c7b0

Вместо
%make DESTDIR=%buildroot install

лучше использовать макрос
%makeinstall_std
Comment 47 Антон Мидюков 2026-01-20 15:52:39 MSK
(Ответ для Danila Skachedubov на комментарий #45)
> Так же обновил пакет cherrytree в таске 405604

Уберите, пожалуйста, Packager из спека. Лучше его не использовать и везде убирать. Так как этот пакет обновляется кем угодно, но не тем, кто там указан.
Comment 48 Антон Мидюков 2026-01-20 15:56:39 MSK
(Ответ для Danila Skachedubov на комментарий #45)
> python3-module-extract-dtb 405640

Не используйте, подалуйста, макрос в лицензии
License: %gpl3only

Спеки читают и люди.
Не забывайте указывать в собираемых пакетах VCS.
Если в changelog пишете с заглавной буквы, то в конце надо ставить точку.
В .gear/rules лишняя пустая строка.
Comment 49 Антон Мидюков 2026-01-20 16:05:18 MSK
(Ответ для Danila Skachedubov на комментарий #45)
> StegoPy 384836

Те же замечания, плюс:

+
+mkdir -p %buildroot%_usr/bin/
+
+cp stegopy.py %buildroot%_usr/bin/
+
+%__chmod +x %buildroot%_usr/bin/stegopy.py

вместо этого:
install -pDm0755 stegopy.py %buildroot%_usr/bin/stegopy

Две лишние строки в конце спека.
Comment 50 Антон Мидюков 2026-01-20 16:10:53 MSK
(Ответ для Danila Skachedubov на комментарий #45)
> Добрый день, собрал и обновил исправленные пакеты libgeopm libgeopmd
> python3-module-geopmpy geopmd в таске 405230
> 
> В пакете freeipa-server-gpo пересмотрел и переделал подход к активации
> плагина и убрал из спека старое решение, собрал в таске 405634
> 
> Так же обновил пакет cherrytree в таске 405604
> 
> Собранные пакеты ранее по следующим таскам
> python3-module-extract-dtb 405640
> StegoPy 384836
> python3-module-vunit 405643

Замечания те же, что и у python3-module-extract-dtb

> spi-tools 384838

Замечания те же.
Comment 51 Danila Skachedubov 2026-01-20 16:36:19 MSK
Спасибо за ревью, хотел бы уточнить пару моментов:
Указывать теги для лицензий было рекомендацией ментора, согласно статье по написанию спек файла, с другой стороны я согласен, что это не совсем читаемо для человека. Для каких случаев тогда применимо использование макросов для лицензий?  
(Ответ для Иван Савин на комментарий #12)
> (Ответ для Danila Skachedubov на комментарий #11)
> Тег License сделать в соответствии с https://www.altlinux.org/Spec#License.

По поводу VCS и URL, в этой же статье указано, что рекомендуется указывать действующий URL домашней страницы проекта, либо если таковой нет — любого другого места, где можно получить архив с исходным кодом. То есть потенциально это могут быть одинаковые ссылки либо же необходимо в таком случае указать в  URL ссылку на страничку с релизами? Например: https://github.com/PabloCastellano/extract-dtb/releases
Спасибо.
Comment 52 Антон Мидюков 2026-01-20 16:42:51 MSK
(Ответ для Danila Skachedubov на комментарий #51)
> Спасибо за ревью, хотел бы уточнить пару моментов:
> Указывать теги для лицензий было рекомендацией ментора, согласно статье по
> написанию спек файла, с другой стороны я согласен, что это не совсем читаемо
> для человека. Для каких случаев тогда применимо использование макросов для
> лицензий?  
> (Ответ для Иван Савин на комментарий #12)
> > (Ответ для Danila Skachedubov на комментарий #11)
> > Тег License сделать в соответствии с https://www.altlinux.org/Spec#License.
> 

Поправил вики на не рекомендуется.
Моё замечание носит рекомендательный характер.

> По поводу VCS и URL, в этой же статье указано, что рекомендуется указывать
> действующий URL домашней страницы проекта, либо если таковой нет — любого
> другого места, где можно получить архив с исходным кодом. То есть
> потенциально это могут быть одинаковые ссылки либо же необходимо в таком
> случае указать в  URL ссылку на страничку с релизами? Например:
> https://github.com/PabloCastellano/extract-dtb/releases
> Спасибо.

В URL нужно стараться указывать сайт с информацией о проекте при наличии.
VCS нужно указывать всегда при наличии, даже если дублируется.
Comment 53 Danila Skachedubov 2026-01-21 12:49:48 MSK
Добрый день, спасибо за такое оперативное ревью и апрув тасок!
При сборке исправленной версии spi-tools в сизиф обнаружил, что данный пакет уже обновили.
Почистил старые таски, так как стал уже сам путаться в них. Пересобираю в одной таске удаляя/добавляя субтаски.
Итого, осталось две таски:
python3-module-vunit 405643
python3-module-extract-dtb 405640
Comment 54 Антон Мидюков 2026-01-21 13:17:07 MSK
https://git.altlinux.org/tasks/405640/gears/200/git?p=git;a=commitdiff;h=b73d8d867448cc1a2021420e74b6d4935d639480

BuildRequires(pre): rpm-build-python3 rpm-build-gir

В pre нужно помещать только пакеты с макросами необходимые для сборки srpm.

BuildRequires: rpm-build-compat >= 1.2

А это вам зачем?
Comment 55 Danila Skachedubov 2026-01-21 13:54:13 MSK
Если честно затрудняюсь ответить, так как собирал этот пакет 2,5 года назад, подозреваю, что эта зависимость была автоматически добавлена при попытке собрать пакет инструментом  rpmbph. Сейчас вижу, что эти зависимости не нужны. Исправил
Comment 56 Антон Мидюков 2026-01-21 13:58:22 MSK
(Ответ для Danila Skachedubov на комментарий #55)
> Если честно затрудняюсь ответить, так как собирал этот пакет 2,5 года назад,
> подозреваю, что эта зависимость была автоматически добавлена при попытке
> собрать пакет инструментом  rpmbph. Сейчас вижу, что эти зависимости не
> нужны. Исправил

Актуализируйте дату в changelog, пожалуйста.

Но в целом, думаю, вы набрались за эти 2,5 года достаточно опыта и готовы к самостоятельной сборке пакетов.
Comment 57 Антон Мидюков 2026-02-03 11:19:59 MSK
Кандидат переведён на 5.0.
Comment 58 Gleb F-Malinovskiy 2026-02-05 13:09:14 MSK
Создана учётная запись ALT Team ID.
Пользователь добавлен в группу мейнтейнеров.

Желаю удачного мейнтейнерства!
Comment 59 Danila Skachedubov 2026-02-05 13:35:14 MSK
Спасибо большое за помощь и наставничество всем участникам!