Created attachment 14630 [details] публичный ssh ключ псевдоним: proskur@altlinux.org почта: proskurinov@basealt.ru имя ментора: Сергей Турчин zerg@altlinux.org Намерен научиться собирать пакеты.
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 снова.
Пользователь добавлен в группу мейнтейнеров. Желаю удачного мейнтейнерства!
Всем спасибо!
