Bug 47805

Created attachment 14631 [details]
gpg публичный ключ

данные для gpg ключа:
Oleg Proskurin <proskur@altlinux.org>

Подтверждаю. Согласен быть ментором.

Собран пакет librum - электронная читалка для книг с бесплатным облаком для их хранения
https://github.com/Librum-Reader/Librum
11 из 20 submodule проекта заменены на системные библиотеки
Добавлены патчи для перехода с версии qt6.5 на 6.4
Добавлен патч для отключения функции "free books" - upstream работает над устранением бага
Пробую отключить сторонний libjpeg

Прошу перевести кандидата на 3.х

Ключи в порядке.

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

T/J/S -> 2.3.

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

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

T/J/S -> 3.6.

подопечный готов отправлять пакеты в Сизиф

up

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

T/J/S -> 4.2.

librum-reader

%ghost %_libdir/libadapters.so
%ghost %_libdir/libapplication.so
%ghost %_libdir/libdomain.so
%ghost %_libdir/libinfrastructure.so
%ghost %_libdir/libpresentation.so

Это очень странное решение для не опакечивания неверсионированных симлинков библиотек, когда не делаешь devel подпакет.
Их нужно в %install удалить и не опакечивать.

Раз всё равно исправлять пакет, ещё несколько лайт-замечаний:
1. В Summary не надо ставить точку в конце.
2. В %description напротив нужно точку в конце предложений ставить. А начинать предложения с заглавной буквы.
3. Перед %description для лучшей читаемости спека нужно отступать пустую строку.
4. Requires: %name-lib - на подпакеты нужно указывать зависимости строго:
Requires: %name-lib = %EVR

EVR - Epoch Version Release.

5. License: GPLv3
Указывать надо в SPDX формате. Все лицензии в /usr/share/license/GPLv3, лицензии SPDX не являются симлинками. Поэтому определить SPDX формат можно так:
$ readlink /usr/share/license/GPLv3
GPL-3.0-only

6. Строки в спеке не должны быть длиннее 80 символов.

7. Помимо URL, нужно указывать VCS, даже если они совпадают.

https://git.altlinux.org/gears/a/alterator-usbguard.git?p=alterator-usbguard.git;a=blob;f=.gear/alterator-usbguard.spec;h=e708c5fe053c02e079ab69cd8433b6c211691e2a;hb=a519d3138ec3329cd8a0c1791d9555dcd9b29059

Небольшие замечания по спеку:
1. Лучше не использовать макрос лицензии
License: %gpl2plus

И сразу писать нормальный SPDX идентификатор лицензии. rpm-build-licenses станет не нужен.

2. rpm-macros-cmake, как и любые другие пакеты rpm-macros-*, стоит помещать в
BuildRequires(pre):

3. %lang(ru)  %_datadir/locale/ru/LC_MESSAGES/alterator-usbguard.mo
Вместо этого лучше в %install:
%find_lang alterator-usbguard

и
%files -f alterator-usbguard.lang

Мало ли, добавятся ещё локали. Не надо будет спек менять.

https://packages.altlinux.org/ru/sisyphus/srpms/handbrake/specfiles/

1. %_datadir/locale/*/LC_MESSAGES/*.mo

тут вы даже не отметили локали, как локали.
В предыдущем комментарии я написал, как правильно делать.


2. Не используете макрос %_iconsdir:
%_datadir/icons/hicolor/scalable/apps/fr.handbrake.ghb.svg

3. Не хватает VCS.

4. Что за проблему этим решили:
%ifarch %ix86
%set_verify_elf_method textrel=relaxed
%endif

?

5. Кэш ни в коем случае пакетить нельзя:
%_datadir/applications/mimeinfo.cache

https://packages.altlinux.org/ru/sisyphus/srpms/surguch/specfiles/

1. Отсутствует URL
2. Отсутствует VCS
3. Не используются макросы %_desktopdir %_iconsdir %_pixmapsdir
4. Исправьте, пожалуйста, багу 57232.

На этом всё

(Ответ для Антон Мидюков на комментарий #16)
> https://packages.altlinux.org/ru/sisyphus/srpms/surguch/specfiles/
> 
> 1. Отсутствует URL
> 2. Отсутствует VCS
> 3. Не используются макросы %_desktopdir %_iconsdir %_pixmapsdir
> 4. Исправьте, пожалуйста, багу 57232.
> 
> На этом всё

Жду исправления замечаний.

Понял вас, буду исправлять в порядке ваших комментариев, по мере прогресса буду отписываться.

Librum-reader
 
- Директива %ghost  заменена на rm -f 

%summary 
удалены точки в конце предлложения

%description
Предложения начинаются с заглавной буквы и заканчиваются точкой.
Добавлена пустая строка перед разделом

Requires:
%name-lib заменено на %name-lib = %EVR

License:
GPLv3 заменена на GPL-3.0-only (https://spdx.org/licenses/)

Добавлен тэг VSC

Перенесены длинные строки.

alterator-usbguard

Добавлен тэг VCS

Ограничение длины строки (80)

License: %gpl2plus -> License: 	GPL-2.0-or-later

rpm-build-licenses - удалено

rpm-macros-cmake - перенесено в BuildRequires(pre)

%lang(ru)  %_datadir/locale/ru/LC_MESSAGES/alterator-usbguard.mo
заменено на

%find_lang alterator-usbguard + %files -f alterator-usbguard.lang

Surguch

* %_desktopdir + %_iconsdir %_pixmapsdir - прокинуты в cmake из .spec
* URL + VCS добавлены
* Длина строк (80)
* Точка в конце summary (удалена)
+ _iconsdir + _pixmapsdir + _desktopdir используются
* BuildRequires(pre): rpm-macros-qt6

Посмотрел исправления замечаний.
Считаю, что кандидат имеет достаточно опыта для самостоятельной сборки пакетов.

handbrake

* собрана новая версия

* %find_lang + %files -f hfb.lang

* Добавлен тэг VCS

* BuildRequires(pre): rpm-macros-cmake rpm-macros-make

* Ограничение длины строки (80)

* Удален applications/mimeinfo.cache

* Использование %_iconsdir

* %set_verify_elf_method textrel=relaxed
было падение build-a на i586 с ошибкой
 TEXTREL entry found: 0x00000000
В проекте под 2млн строк, fpic вроде как везде есть где я проверил,
проводить расследование только ради i586 - жалко времени,я считаю, что кодирование
видео на этих системах всё равно занятие бесполезное.

Мне посоветовали или отключить сборку на i586 или отключить проверку,
я отключил проверку - люди активно пользуются пакетом (врядли на таких древних машинах), проблем не было.
Хотя, наверное, лучше вообще выключить сборку на i586.

(Ответ для Антон Мидюков на комментарий #22)
> Посмотрел исправления замечаний.
> Считаю, что кандидат имеет достаточно опыта для самостоятельной сборки
> пакетов.

Произошла коллизия. Oleg Proskurin, в таких ситуациях выбирайте "Сохранить только мой комментарий", чтобы не ломать изменения бага от другого участника.
Поправил аннотацию на 5.0 снова.

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

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

Всем спасибо!