Bug 51153

Created attachment 16624 [details]
Ключ ssh

(In reply to minergenon from comment #0)
> Created attachment 16623 [details]

Это не формальный критерий, но комментарий в gpg-ключе лучше никакой не делать, чем такой.

T/J/S -> 1.3.

Ой, отклика ментора же ещё не было.

(Ответ для Gleb F-Malinovskiy на комментарий #3)
> Ой, отклика ментора же ещё не было.
Подтверждаю согласие быть ментором.

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

> Это не формальный критерий, но комментарий в gpg-ключе лучше никакой не
> делать, чем такой.
Теперь это формальное требование.
Приложите, пожалуйста, ключ без комментария.

Created attachment 16963 [details]
Ключ ssh

Created attachment 16964 [details]
Ключ gpg

Comment on attachment 16963 [details]
Ключ ssh

Так, этот тот же, что был.

(In reply to minergenon from comment #8)
> Created attachment 16964 [details]
> Ключ gpg
И этот тот же.

(Ответ для Gleb F-Malinovskiy на комментарий #10)
> (In reply to minergenon from comment #8)
> > Created attachment 16964 [details] [подробности] [details]
> > Ключ gpg
> И этот тот же.

Не каких комментариев не делаю только краткое описание

(In reply to minergenon from comment #11)
> Не каких комментариев не делаю только краткое описание

Имеется в виду комментарий внутри gpg-ключа.  Это одно из полей, которое задаётся при создании ключа.

Created attachment 16965 [details]
Ключ gpg

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

T/J/S -> 2.3.

Отправил пакеты:
To gitery:/people/minergenon/packages/sdl2-gamepad-mapper.git
To gitery:/people/minergenon/packages/libopenrazer.git
To gitery:/people/minergenon/packages/razergenie.git

Отправил пакеты:
To gitery:/people/minergenon/packages/xplane-sdk.git
To gitery:/people/minergenon/packages/hydrajoy.git
To gitery:/people/minergenon/packages/opentrack.git

Отправил пакеты:
To gitery:/people/minergenon/packages/OpenComposite.git
To gitery:/people/minergenon/packages/xr-hardware.git

Отправил пакет:
To gitery:/people/minergenon/packages/openhmd.git

Отправил пакет:
To gitery:/people/minergenon/packages/scplot.git

Отправил пакет:
To gitery:/people/minergenon/packages/librealsense.git

Отправил пакеты:
To gitery:/people/minergenon/packages/qvr.git
To gitery:/people/minergenon/packages/bino.git

Отправил пакет:
To gitery:/people/minergenon/packages/monado.git

Отправил пакеты:
To gitery:/people/minergenon/packages/gulkan.git
To gitery:/people/minergenon/packages/gxr.git
To gitery:/people/minergenon/packages/sphvr.git

(Ответ для minergenon на комментарий #16)
> Отправил пакеты:
> To gitery:/people/minergenon/packages/xplane-sdk.git
> To gitery:/people/minergenon/packages/hydrajoy.git
> To gitery:/people/minergenon/packages/opentrack.git

Добрый день! Ой, что-то время пролетает...
Общие замечания:

Добавление в спек требует серьёзного обоснования. Как минимум комментария, почему это понадобилось.
%set_verify_elf_method skip

Например, вы так спрятали RPATH
verify-elf: WARNING: ./usr/lib64/opentrack/opentrack-tracker-test.so: RPATH entry found: /usr/lib64/opentrack

Например, так вы спрятали то, что в hydrajoy лежит бинарная библиотека sixense. Вы должны собрать её из исходников в отдельном пакете, а не паковать из этого.

Указывать в сборочных зависимостях glibc-devel не нужно

Такая конструкция запакует только файлы в xplane_sdk, но не сам каталог
%_includedir/xplane_sdk/*
обычно пишется так:
%_includedir/xplane_sdk/

Если для пакета известен URL с git-репозиторием, то стоит указать после тэга Url:
VCS: URL

Выходной пакет xplane-sdk я бы назвал xplane-sdk-devel, чтобы все пакеты для разработки в репозитории оканчивались на devel. Но я вижу, что у нас есть несколько таких, которые оканчиваются просто на sdk (xorg-sdk, dotnet-sdk, plasma-sdk), поэтому может и перебор.

Кандидат готов собирать пакеты.

(Ответ для minergenon на комментарий #18)
> Отправил пакет:
> To gitery:/people/minergenon/packages/openhmd.git

На выходе должны быть
libopenhmd и libopenhmd-devel

И файл библиотеки лучше назвать в соответствии с
https://www.altlinux.org/Shared_Libs_Policy

(Ответ для minergenon на комментарий #19)
> Отправил пакет:
> To gitery:/people/minergenon/packages/scplot.git

Нужно убрать rpm-macros-fedora-compat

Замечания исправил.
Пакет hydrajoy дальше личного репозитория не уйдет (эксперементальная версия и врят ли уже дальше будет поддерживаться)
Отправил пакеты:
To gitery:/people/minergenon/packages/wlx-overlay-s.git
To gitery:/people/minergenon/packages/g3k.git
To gitery:/people/minergenon/packages/libinputsynth.git
To gitery:/people/minergenon/packages/xrdesktop.git

Отправил пакеты:
To gitery:/people/minergenon/packages/python3-module-pulsemixer.git
To gitery:/people/minergenon/packages/python3-module-pykickstart.git
To gitery:/people/minergenon/packages/python3-module-setuptools-git-versioning.git

Отправил пакеты:
To gitery:/people/minergenon/packages/ccsum.git
To gitery:/people/minergenon/packages/inputplumber.git
To gitery:/people/minergenon/packages/nping.git
To gitery:/people/minergenon/packages/thoth.git

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

T/J/S -> 3.6.

Отправлены пакеты в сборочницу:
sisyphus: #372664
sisyphus: #372882
sisyphus: #372885
sisyphus: #372889
sisyphus: #372896
sisyphus: #372877
sisyphus: #372957
sisyphus: #372878
sisyphus: #372958
sisyphus: #372962
sisyphus: #372964
sisyphus: #372970
sisyphus: #372972

1.
 https://packages.altlinux.org/ru/tasks/372664/

Для модулей python зависимости находятся автоматически, поэтому явно указывать
Requires: python3-module-cffi
смысла нет, только повторение получается:

$ epm req python3-module-vulkan
 $ rpm -q --requires python3-module-vulkan
python3-module-cffi
vulkan
/usr/lib/python3/site-packages
python3(_cffi_backend) < 0
python3(cffi) < 0
...

python3-devel нужен только для компиляции, в noarch пакете этого не может быть, и он не нужен.

Я бы предложил ещё разными строчками писать пакеты setuptools/wheel (инструменты сборки)  и остальные:

 BuildRequires(pre): rpm-build-python3
-BuildRequires: python3-devel python3-module-setuptools python3-module-wheel python3-module-cffi
-Requires: python3-module-cffi
+BuildRequires: python3-module-setuptools python3-module-wheel
+BuildRequires: python3-module-cffi
+
 Requires: vulkan

С
 Requires: vulkan
интересно получается, но как будто это самый лучший вариант, учитывая, что сейчас подгружается libvulkan,so.1 из пакета libvulkan1

3.
https://packages.altlinux.org/ru/tasks/372885/

Не хорошо упаковывать каталог
%python3_sitelibdir/pycache/
наверное, имелось в виду
%python3_sitelibdir/pycache/*

Поскольку для работы пакета будет требоваться git, лучше добавить
Requires: git-core

и в BuildRequires лучше git-core указать, а не просто git.
Так делают 180 пакетов из 210, которые имеют зависимость на git.

9.
  https://packages.altlinux.org/ru/tasks/372958/

С qvr всё отлично, но основной пакет должен называться libqvr, к нему libqvr-devel и libqvr-devel-doc
Проще всего это сделать, если исходный пакет сразу назвать libqvr

В bino зависимость
Requires: qvr
не нужна, сама найдётся

Замечания подправил

Собраны новые пакеты:
https://packages.altlinux.org/ru/tasks/376768/   - sixpair
https://packages.altlinux.org/ru/tasks/376792/   - joycond
https://packages.altlinux.org/ru/tasks/376872/   - ckb-next
https://packages.altlinux.org/ru/tasks/376878/   - joystick-monitor
https://packages.altlinux.org/ru/tasks/376918/   - libfreenect

Пакеты уже в Sisyphus:
https://packages.altlinux.org/ru/tasks/376642/   -libqvr, bino
https://packages.altlinux.org/ru/tasks/372962/   -libopenrazer, razergenie
https://packages.altlinux.org/ru/tasks/372957/   -python3-module-pulsemixer
https://packages.altlinux.org/ru/tasks/372878/   -python3-module-RecoverPy
https://packages.altlinux.org/ru/tasks/372882/   -python3-module-pykickstart
https://packages.altlinux.org/ru/tasks/372885/   -python3-module-setuptools-git-versioning
https://packages.altlinux.org/ru/tasks/372664/   -python3-module-vulkan

Отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/374405/   - librealsense
https://packages.altlinux.org/ru/tasks/374403/   - monado-vulkan-layers

Собран новый пакет:
https://packages.altlinux.org/ru/tasks/378648/   - ludusavi

Собранs новые пакеты:
https://packages.altlinux.org/ru/tasks/378752/   - spacenavd, spnavcfg

Отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/374408/   - libsurvive

Апрувлено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/379763/ - sciplot
https://packages.altlinux.org/ru/tasks/374975/ - xrizer
https://packages.altlinux.org/ru/tasks/375021/ - motoc
https://packages.altlinux.org/ru/tasks/374410/ - OpenComposite

Одобрен и отправлен в Sisyphus:
https://packages.altlinux.org/ru/tasks/375796/ - vdu_controls

Одобрено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/375827/ - steamtinkerlaunch
https://packages.altlinux.org/ru/tasks/375614/ - nfpm

Одобрено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/375988/ - Decker
https://packages.altlinux.org/ru/tasks/376127/ - GPU-Viewer

Одобрено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/372889/ - ccsum
https://packages.altlinux.org/ru/tasks/376872/ - ckb-next
https://packages.altlinux.org/ru/tasks/379069/ - envision
https://packages.altlinux.org/ru/tasks/375959/ - game-devices-udev
https://packages.altlinux.org/ru/tasks/376792/ - joycond

Одобрено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/376878/ - joystick-monitor
https://packages.altlinux.org/ru/tasks/376918/ - libfreenect
https://packages.altlinux.org/ru/tasks/378648/ - ludusavi
https://packages.altlinux.org/ru/tasks/375880/ - new-lg4ff
1https://packages.altlinux.org/ru/tasks/376180/ - qpdftools
1https://packages.altlinux.org/ru/tasks/378752/ - spacenavd|spnavcfg
https://packages.altlinux.org/ru/tasks/387256/ - monado

Одобрено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/372964/  - thoth
https://packages.altlinux.org/ru/tasks/375855/  - protopedal
https://packages.altlinux.org/ru/tasks/372877/  - nping

Одобрено и отправлено в Sisyphus:
https://packages.altlinux.org/ru/tasks/376768/  - sixpair
https://packages.altlinux.org/ru/tasks/376157/  - python3-module-hwdata|tuxclocker

Собрано множество пакетов.
Считаю, что кандидат готов к самостоятельной сборке пакетов в ALT Sisyphus. Прошу назначить ревьювера.

Обновил до актуальных. Прошу дать апрув.
nping - https://packages.altlinux.org/ru/tasks/401710/
python3-module-hwdata - https://packages.altlinux.org/ru/tasks/401712/
thoth - https://packages.altlinux.org/ru/tasks/401713/

Согласно новым правилам:
https://www.altlinux.org/Team/Join/Review_Team

с согласия других членов команды Review_Team беру кандидата на рецензирование.
Прошу кандидата обращаться за аппрувом тасков сразу ко мне antohami@.
По мере ревью ранее сделанных тасков буду отписываться сюда.

К nping и thoth аналогичные замечания:

1. В BuildRequires(pre) нужно помещать только пакеты с макросами, которые нужны для формирования srpm.

2. Почему не используете %rust_prep ? Читали:
https://www.altlinux.org/RPM/Rust
?

3. BuildRequires: /proc
для чего оказался нужен?

4. URL, как и любую другую аббревиатуру, следует писать заглавными буквами.

python3-module-hwdata

>BuildRequires(pre): rpm-build-python3

Повторюсь: в BuildRequires(pre) нужно помещать только пакеты с макросами, которые нужны для формирования srpm.

tuxclocker

1. %_libdir/libtuxclocker.so
Библиотека без версионирования, devel пакета для неё нет.
Её следует поместить в %_libdir/tuxclocker/

2. %_datadir/applications
Следует использовать макрос %_desktopdir

3. нужно убрать зависимость на libnvidia-ml, так как это теперь чисто для devel пакетов нужен и тянет за собой драйвера nvidia (zerg исправил уже).

sixpair

Не используете флаги компиляции дефолтные - %optflags:
gcc -o sixpair %SOURCE0 -lusb
gcc -o sixhidtest %SOURCE1
gcc -o xsixhidtest %SOURCE2 -lX11 -lm

Добавьте.

monado

1. Зачем включили %set_verify_elf_method relaxed ?
Это нужно делать в крайнем случае отдельным коммитом с подробным обоснованием причин.

2. Зачем понадобился BuildRequires: /proc ?

3. Зачем эти зависимости:
Requires: libopenhmd
Requires: librealsense
Requires: libsurvive
?

Над ними очень странный комментарий:
+# Missing dependencies for the build.
+# depthai  LeapV2 LeapSDK ONNXRuntime Percetto

Зачем в рантайме у пользователя что-то собирать?

4. Вместо макроса %cmakeinstall_std следует использует %cmake_install

5. %_libdir/libmonado.so.*
В соответствии с Shared Libs Policy вам следует выделить отдельный подпакет с библиотекой:
https://www.altlinux.org/Shared_Libs_Policy

6. %_libdir/libopenxr_monado.so
Библиотеку без версионирования следует поместить в подкаталог %_libdir/monado/

spacenavd

Зачем пришлось указывать:
Requires: xdpyinfo
?

libfreenect

1. следует привести в соответствие с:
https://www.altlinux.org/Shared_Libs_Policy

2. Объяснить, в чём затык сборки на других аппаратных платформах

3. %cmakeinstall_std заменить на %cmake_install

4. find -type f -exec grep -l '#!/usr/bin/env python' {} \; | xargs sed -i 's|#!/usr/bin/env python|#!/usr/bin/env python3|g'

такие длинные строки следует разбивать на несколько <= 80 символов, для лучшего чтения.
Менять следует на %__python3

(Ответ для Антон Мидюков на комментарий #59)

пересобрал
sisyphus: #418472 nbping.git=0.6.1-alt1
sisyphus: #418482 thoth.git=0.1.88-alt1
Не могу понять почему сборка завершается с ошибкой sisyphus: #418504 sixpair.git=1.8-alt2

Особой информации по сборки python пакетов не нашел но если верить шаблонам из /usr/share/spectemplates/ то используется BuildRequires(pre): rpm-build-python3

По поводу как использовать Url или URL тоже особой информации нет и если верить этому https://www.altlinux.org/Spec#%25install или этому https://www.altlinux.org/Технология_сборки_пакетов_RPM то в примере указан Url, если посмотреть на спек файлы здесь https://github.com/altlinux/specs/ то тут тоже вопрос.
Откуда начинающему ментейнеру понять как собирать если даже в документации не понять как должно быть правильно.
Если есть какие то конкретные замечания готов выслушать.
Если есть где то более правильная документация готов изучить и пересобрать все свои пакеты заново.

(Ответ для minergenon на комментарий #60)
> (Ответ для Антон Мидюков на комментарий #59)
> 
> пересобрал
> sisyphus: #418472 nbping.git=0.6.1-alt1

В %description очень длинная строка. Нужно разбивать строки в спеке по 80 символов.
В коммите спек добавляется целиком, при этом в changelog это третья версия. Возникает вопрос: как так? Зачем было убирать предыдущие коммиты? Нужно или их возвращать или changelog чистить.

> sisyphus: #418482 thoth.git=0.1.88-alt1

Тут всё нормально. Одобрил.

> Не могу понять почему сборка завершается с ошибкой sisyphus: #418504
> sixpair.git=1.8-alt2
> 

Вы перезаписали коммит, который уже попал в /gears/s/sixpair.git, нарушили проверку наследования из этого git. Так делать нельзя без острой необходимости.

> Особой информации по сборки python пакетов не нашел но если верить шаблонам
> из /usr/share/spectemplates/ то используется BuildRequires(pre):
> rpm-build-python3
> 

Когда-то давно не было пакета rpm-macros-python3, поэтому надо понемногу это выправлять. Это вам не в укор. Но есть же ментор :)

> По поводу как использовать Url или URL тоже особой информации нет и если
> верить этому https://www.altlinux.org/Spec#%25install или этому
> https://www.altlinux.org/Технология_сборки_пакетов_RPM то в примере указан
> Url, если посмотреть на спек файлы здесь https://github.com/altlinux/specs/
> то тут тоже вопрос.
> Откуда начинающему ментейнеру понять как собирать если даже в документации
> не понять как должно быть правильно.
> Если есть какие то конкретные замечания готов выслушать.
> Если есть где то более правильная документация готов изучить и пересобрать
> все свои пакеты заново.

Пересобирать только ради косметики ничего не нужно. На косметику указываю только при наличии более серьёзных замечаний.
Я знаю, что на вики много всего устаревшего. Но есть ментор, и есть знания из русского и английского языков.

(Ответ для Антон Мидюков на комментарий #61)
> (Ответ для minergenon на комментарий #60)
> > (Ответ для Антон Мидюков на комментарий #59)
> > 
> > пересобрал
> > sisyphus: #418472 nbping.git=0.6.1-alt1
> 
> В %description очень длинная строка. Нужно разбивать строки в спеке по 80
> символов.
> В коммите спек добавляется целиком, при этом в changelog это третья версия.
> Возникает вопрос: как так? Зачем было убирать предыдущие коммиты? Нужно или
> их возвращать или changelog чистить.

%description укоротил. Вы точно о коммитах этого пакета говорите?
 
> > Не могу понять почему сборка завершается с ошибкой sisyphus: #418504
> > sixpair.git=1.8-alt2
> > 
> 
> Вы перезаписали коммит, который уже попал в /gears/s/sixpair.git, нарушили
> проверку наследования из этого git. Так делать нельзя без острой
> необходимости.

И как исправлять данный случай?

> > Особой информации по сборки python пакетов не нашел но если верить шаблонам
> > из /usr/share/spectemplates/ то используется BuildRequires(pre):
> > rpm-build-python3
> > 
> 
> Когда-то давно не было пакета rpm-macros-python3, поэтому надо понемногу это
> выправлять. Это вам не в укор. Но есть же ментор :)

Так все таки как правильно должно быть? Что и как должно быть указано?
Ментор на этом внимания не заострял, да и аппрувил сам же.

> > По поводу как использовать Url или URL тоже особой информации нет и если
> > верить этому https://www.altlinux.org/Spec#%25install или этому
> > https://www.altlinux.org/Технология_сборки_пакетов_RPM то в примере указан
> > Url, если посмотреть на спек файлы здесь https://github.com/altlinux/specs/
> > то тут тоже вопрос.
> > Откуда начинающему ментейнеру понять как собирать если даже в документации
> > не понять как должно быть правильно.
> > Если есть какие то конкретные замечания готов выслушать.
> > Если есть где то более правильная документация готов изучить и пересобрать
> > все свои пакеты заново.
> 
> Пересобирать только ради косметики ничего не нужно. На косметику указываю
> только при наличии более серьёзных замечаний.
> Я знаю, что на вики много всего устаревшего. Но есть ментор, и есть знания
> из русского и английского языков.

Итог.
Что нужно поправить?

(Ответ для minergenon на комментарий #62)
> (Ответ для Антон Мидюков на комментарий #61)
> > (Ответ для minergenon на комментарий #60)
> > > (Ответ для Антон Мидюков на комментарий #59)
> > > 
> > > пересобрал
> > > sisyphus: #418472 nbping.git=0.6.1-alt1
> > 
> > В %description очень длинная строка. Нужно разбивать строки в спеке по 80
> > символов.
> > В коммите спек добавляется целиком, при этом в changelog это третья версия.
> > Возникает вопрос: как так? Зачем было убирать предыдущие коммиты? Нужно или
> > их возвращать или changelog чистить.
> 
> %description укоротил. Вы точно о коммитах этого пакета говорите?
>  

А как объяснить иначе наличие нескольких записей в changelog при одном коммите:
https://git.altlinux.org/tasks/418472/gears/40/git?p=git;a=commitdiff;h=a29ddb9eda5b691421cb33124e313f500243e954
?

> > > Не могу понять почему сборка завершается с ошибкой sisyphus: #418504
> > > sixpair.git=1.8-alt2
> > > 
> > 
> > Вы перезаписали коммит, который уже попал в /gears/s/sixpair.git, нарушили
> > проверку наследования из этого git. Так делать нельзя без острой
> > необходимости.
> 
> И как исправлять данный случай?

перенести изменения на /gears/s/sixpair.git ветки sisyphus, не трогая коммиты в ветке sisyphus.

> 
> > > Особой информации по сборки python пакетов не нашел но если верить шаблонам
> > > из /usr/share/spectemplates/ то используется BuildRequires(pre):
> > > rpm-build-python3
> > > 
> > 
> > Когда-то давно не было пакета rpm-macros-python3, поэтому надо понемногу это
> > выправлять. Это вам не в укор. Но есть же ментор :)
> 
> Так все таки как правильно должно быть? Что и как должно быть указано?

достаточно:

BuildRequires: rpm-build-python3

rpm-macros-python3 входит в базовую сборочную среду.

> Ментор на этом внимания не заострял, да и аппрувил сам же.

Я это понимаю.

> 
> > > По поводу как использовать Url или URL тоже особой информации нет и если
> > > верить этому https://www.altlinux.org/Spec#%25install или этому
> > > https://www.altlinux.org/Технология_сборки_пакетов_RPM то в примере указан
> > > Url, если посмотреть на спек файлы здесь https://github.com/altlinux/specs/
> > > то тут тоже вопрос.
> > > Откуда начинающему ментейнеру понять как собирать если даже в документации
> > > не понять как должно быть правильно.
> > > Если есть какие то конкретные замечания готов выслушать.
> > > Если есть где то более правильная документация готов изучить и пересобрать
> > > все свои пакеты заново.
> > 
> > Пересобирать только ради косметики ничего не нужно. На косметику указываю
> > только при наличии более серьёзных замечаний.
> > Я знаю, что на вики много всего устаревшего. Но есть ментор, и есть знания
> > из русского и английского языков.
> 
> Итог.
> Что нужно поправить?

Те пакеты, по которым я сделал замечания.

В таске 418472 коммиты "rename spec: nping → nbping, update rules and tags for 0.6.1" и "update vendored cargo modules with cargo vendor for 0.6.1" должны быть уже после Merge tag 'v0.6.1'. Коммит 0.6.1-alt1 пустой, что странно.

Доброго времени суток.

Я прошу прощение, но я хотел бы задать вопрос Sergey Palcheh, учитывая, что Вы начали работу с monado, вы планируете продолжить сопровождение пакета monado дальше?

так как я его подобрал сломанный и nobody, мне интересны дальнейшие Ваши планы, впрочем как и по пакету libsurvive ?

Если да, то я уберу у себя эти два пакета с отслеживания.

С уважением.

(Ответ для Aleksandr Shamaraev на комментарий #65)
> Доброго времени суток.
> 
> Я прошу прощение, но я хотел бы задать вопрос Sergey Palcheh, учитывая, что
> Вы начали работу с monado, вы планируете продолжить сопровождение пакета
> monado дальше?
> 
> так как я его подобрал сломанный и nobody, мне интересны дальнейшие Ваши
> планы, впрочем как и по пакету libsurvive ?
> 
> Если да, то я уберу у себя эти два пакета с отслеживания.
> 
> С уважением.
Планирую продолжать сопровождение

(Ответ для minergenon на комментарий #66)
> (Ответ для Aleksandr Shamaraev на комментарий #65)
> > Доброго времени суток.
> > 
> > Я прошу прощение, но я хотел бы задать вопрос Sergey Palcheh, учитывая, что
> > Вы начали работу с monado, вы планируете продолжить сопровождение пакета
> > monado дальше?
> > 
> > так как я его подобрал сломанный и nobody, мне интересны дальнейшие Ваши
> > планы, впрочем как и по пакету libsurvive ?
> > 
> > Если да, то я уберу у себя эти два пакета с отслеживания.
> > 
> > С уважением.
> Планирую продолжать сопровождение

Хорошо

(Ответ для Антон Мидюков на комментарий #63)
> (Ответ для minergenon на комментарий #62)
> > (Ответ для Антон Мидюков на комментарий #61)
> > > (Ответ для minergenon на комментарий #60)
> > > > (Ответ для Антон Мидюков на комментарий #59)
> > > > 
> > > > пересобрал
> > > > sisyphus: #418472 nbping.git=0.6.1-alt1
> > > 
> > > В %description очень длинная строка. Нужно разбивать строки в спеке по 80
> > > символов.
> > > В коммите спек добавляется целиком, при этом в changelog это третья версия.
> > > Возникает вопрос: как так? Зачем было убирать предыдущие коммиты? Нужно или
> > > их возвращать или changelog чистить.
> > 
> > %description укоротил. Вы точно о коммитах этого пакета говорите?
> >  
> 
> А как объяснить иначе наличие нескольких записей в changelog при одном
> коммите:
> https://git.altlinux.org/tasks/418472/gears/40/git?p=git;a=commitdiff;
> h=a29ddb9eda5b691421cb33124e313f500243e954
> ?
> 
> > > > Не могу понять почему сборка завершается с ошибкой sisyphus: #418504
> > > > sixpair.git=1.8-alt2
> > > > 
> > > 
> > > Вы перезаписали коммит, который уже попал в /gears/s/sixpair.git, нарушили
> > > проверку наследования из этого git. Так делать нельзя без острой
> > > необходимости.
> > 
> > И как исправлять данный случай?
> 
> перенести изменения на /gears/s/sixpair.git ветки sisyphus, не трогая
> коммиты в ветке sisyphus.
> 
> > 
> > > > Особой информации по сборки python пакетов не нашел но если верить шаблонам
> > > > из /usr/share/spectemplates/ то используется BuildRequires(pre):
> > > > rpm-build-python3
> > > > 
> > > 
> > > Когда-то давно не было пакета rpm-macros-python3, поэтому надо понемногу это
> > > выправлять. Это вам не в укор. Но есть же ментор :)
> > 
> > Так все таки как правильно должно быть? Что и как должно быть указано?
> 
> достаточно:
> 
> BuildRequires: rpm-build-python3
> 
> rpm-macros-python3 входит в базовую сборочную среду.
> 
> > Ментор на этом внимания не заострял, да и аппрувил сам же.
> 
> Я это понимаю.
> 
> > 
> > > > По поводу как использовать Url или URL тоже особой информации нет и если
> > > > верить этому https://www.altlinux.org/Spec#%25install или этому
> > > > https://www.altlinux.org/Технология_сборки_пакетов_RPM то в примере указан
> > > > Url, если посмотреть на спек файлы здесь https://github.com/altlinux/specs/
> > > > то тут тоже вопрос.
> > > > Откуда начинающему ментейнеру понять как собирать если даже в документации
> > > > не понять как должно быть правильно.
> > > > Если есть какие то конкретные замечания готов выслушать.
> > > > Если есть где то более правильная документация готов изучить и пересобрать
> > > > все свои пакеты заново.
> > > 
> > > Пересобирать только ради косметики ничего не нужно. На косметику указываю
> > > только при наличии более серьёзных замечаний.
> > > Я знаю, что на вики много всего устаревшего. Но есть ментор, и есть знания
> > > из русского и английского языков.
> > 
> > Итог.
> > Что нужно поправить?
> 
> Те пакеты, по которым я сделал замечания.

Поправил. Надеюсь правильно.

libfreenect 0.7.5 alt2

1.
# Move python module if cmake installed it to wrong location
if [ -f %buildroot/usr/lib/python3/site-packages/freenect.so ]; then
    if [ "%buildroot/usr/lib/python3/site-packages" != %buildroot%python3_sitelibdir" ]; then
        mv %buildroot/usr/lib/python3/site-packages/freenect.so \
            %buildroot%python3_sitelibdir/
    fi
fi

Чем такое мудрить, лучше устранить причину безобразия. Попробуйте исправить CMakeLists.txt

2. В changelog не отражены важные изменения: пакет libfreenect-opencv переименован в libfreenect-cv, пакет openni переименован в openni2, выделен новый пакет libfreenect-utils.

3. Когда переименовываете пакеты, заботьтесь о корректном обновлении.
В текущем виде никуда не годится. libfreenect-cv должен заменять libfreenect-opencv, openni2 заменять openni.
Но расскажите, чем вызвана необходимость переименования?

monado 25.1.0 alt3

%add_verify_elf_skiplist %_datadir/steamvr-monado/bin/linux64/driver_monado.so

mv %buildroot%_libdir/libopenxr_monado.so %buildroot%_libdir/monado/

Так делать нельзя. Вы отключили проверку, из-за чего в будущем может быть пропущена ошибка сборки библиотеки.
Нужно править CMakeLists.txt

libfreenect 0.7.5 alt2

1. Ошибочно упаковывать сюда:
/usr/share/libfreenect/libfreenectConfig.cmake

Нужно в %_datadir/cmake/Modules и помещать в devel пакет.

2. Ошибочно упаковывать сюда:
/usr/share/libfreenect/fwfetcher.py

Это утилита, предлагаю запаковать как %_bindir/freenect-fwfetcher в пакет utils.

3. Вот это не Shared Lib Policy совсем:
%files
%doc APACHE20 GPL2 README.md CONTRIB
%_libdir/libfreenect.so.0*
%_libdir/libfreenect_sync.so.0*
%_datadir/%name
%_udevrulesdir/51-kinect.rules

%sover должен быть в имени пакета. В пакете не должно быть ничего, что не содержит %sover, так как невозможно будет установить одновременно библиотеку со старым и новым %sover.

Предлагаю выделить новый пакет libfreenect%sover:

%files -n %name%sover
%_libdir/%name.so.%sover
%_libdir/%name.so.%sover.*
%_libdir/%{name}_sync.so.%sover
%_libdir/%{name}_sync.so.%sover.*

и прописать у него конфликт с предыдущей версией libfreenect.

Нужно где то еще что то поправить?

(Ответ для minergenon на комментарий #72)
> Нужно где то еще что то поправить?

libfreenect

>Requires: %name%sover = %EVR

Нужно указывать только devel пакету. Поправьте, пожалуйста, крайний коммит.

_______

monado

>mv %buildroot%_libdir/libopenxr_monado.so %buildroot%_libdir/monado/

Это тоже нужно поправить в CMakeLists.txt

Удалите старый таск 401713

> https://git.altlinux.org/tasks/390383

Предлагаю попробовать самостоятельно исправить все косяки в спеке.

libfreenect

Упустил из виду. Вы конфликт прописали у libfreenect, а нужно у библиотеки, потому что именно она конфликтует по содержимому со старой версией libfreenect.

(Ответ для Антон Мидюков на комментарий #74)
> Удалите старый таск 401713
> 
> > https://git.altlinux.org/tasks/390383
> 
> Предлагаю попробовать самостоятельно исправить все косяки в спеке.

Сделал бы примерно так.

%define _unpackaged_files_terminate_build 1

Name: yuki-iptv
Version: 0.0.19
Release: alt1

Summary: IPTV player with EPG support (Astroncia IPTV fork)
License: GPLv3
Group: Video
URL: https://codeberg.org/liya/yuki-iptv

# Source-url: https://codeberg.org/liya/yuki-iptv/archive/%version.tar.gz
Source: %name-%version.tar

BuildArch: noarch

BuildRequires(pre): rpm-build-python3

Requires: libmpv2 ffmpeg

Requires: python3-module-PyQt6 python3-module-Pillow

# recommends
Requires: yt-dlp

%add_python3_path %_prefix/lib/%name

%add_python3_req_skip gi.repository.GLib
%add_python3_req_skip gi.repository.Gio

%description
IPTV player with EPG support (Astroncia IPTV fork).

Disclaimer: yuki-iptv doesn't provide any playlists or other digital content.
The channels and pictures in the screenshots are for demonstration purposes only.

%prep
%setup
sed -i "s/__DEB_VERSION__/%version/g" usr/lib/yuki-iptv/yuki-iptv.py

%build
%make_build

%install
mkdir -p %buildroot%_prefix
cp -r usr/* %buildroot%_prefix/

%find_lang %name


%files -f %name.lang
%doc README.md
%_bindir/%name
%_prefix/lib/%name/
%_datadir/%name/
%_desktopdir/%name.desktop
%_iconsdir/hicolor/scalable/apps/%name.svg
%_datadir/metainfo/%name.appdata.xml

В данный момент ссылка на загрузку дистрибутива не доступна и дальнейшее обновление пакета под вопросом.

(Ответ для minergenon на комментарий #76)
> (Ответ для Антон Мидюков на комментарий #74)
> > Удалите старый таск 401713
> > 
> > > https://git.altlinux.org/tasks/390383
> > 
> > Предлагаю попробовать самостоятельно исправить все косяки в спеке.
> 
> Сделал бы примерно так.
> 
> %define _unpackaged_files_terminate_build 1
> 
> Name: yuki-iptv
> Version: 0.0.19
> Release: alt1
> 
> Summary: IPTV player with EPG support (Astroncia IPTV fork)
> License: GPLv3

Лицензию, на будущее, указывайте в формате SPDX: GPL-3.0

> Group: Video
> URL: https://codeberg.org/liya/yuki-iptv
> 
> # Source-url: https://codeberg.org/liya/yuki-iptv/archive/%version.tar.gz
> Source: %name-%version.tar
> 
> BuildArch: noarch
> 
> BuildRequires(pre): rpm-build-python3

Напоминаю, что rpm-build-*, как правило не требуется помещать в BuildRequires(pre), достаточно в BuildRequires. В BuildRequires(pre) помещаем только rpm-macros-*.

> 
> Requires: libmpv2 ffmpeg
> 


> Requires: python3-module-PyQt6 python3-module-Pillow

Их не нужно указывать.

> 
> # recommends
> Requires: yt-dlp
> 
> %add_python3_path %_prefix/lib/%name

Добавляя вот так, получаем кучу провайдсов, которые невалидны (импортировать эти модули никто не сможет, в непубличном месте). Нужно отключать поиск python3 provides для пакета:
AutoProv: nopython3

> 
> %add_python3_req_skip gi.repository.GLib
> %add_python3_req_skip gi.repository.Gio

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

> 
> %description
> IPTV player with EPG support (Astroncia IPTV fork).
> 
> Disclaimer: yuki-iptv doesn't provide any playlists or other digital content.
> The channels and pictures in the screenshots are for demonstration purposes
> only.
> 
> %prep
> %setup
> sed -i "s/__DEB_VERSION__/%version/g" usr/lib/yuki-iptv/yuki-iptv.py
> 
> %build
> %make_build
> 
> %install
> mkdir -p %buildroot%_prefix
> cp -r usr/* %buildroot%_prefix/
> 
> %find_lang %name
> 
> 
> %files -f %name.lang
> %doc README.md
> %_bindir/%name
> %_prefix/lib/%name/
> %_datadir/%name/
> %_desktopdir/%name.desktop
> %_iconsdir/hicolor/scalable/apps/%name.svg
> %_datadir/metainfo/%name.appdata.xml
> 
> В данный момент ссылка на загрузку дистрибутива не доступна и дальнейшее
> обновление пакета под вопросом.

Ладно. Неплохо.
Полагаю, кандидат готов к самостоятельной сборке пакетов.

(Ответ для Антон Мидюков на комментарий #73)
> (Ответ для minergenon на комментарий #72)
> > Нужно где то еще что то поправить?

> _______
> 
> monado
> 
> >mv %buildroot%_libdir/libopenxr_monado.so %buildroot%_libdir/monado/
> 
> Это тоже нужно поправить в CMakeLists.txt

Дайте пожалуйста аппрув на monado

Пользователь добавлен в группу мейнтейнеров.

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