Псевдоним: alexvk Пересылка почты: alexvk72@ya.ru Ментор: Григорий Устинов <grenka@altlinux.org> Цель: научиться собирать и сопровождать пакеты, локально собрал темы Yaru, Mint, несколько плагинов для Cinnamon и GNOME.
Created attachment 16893 [details] SSH key
Created attachment 16894 [details] GPG key
Менторство подтверждаю.
Ментор есть, ключи в порядке. T/J/S -> 1.3.
Кандидат готов начать процесс, выдайте, пожалуйста, сборочницу.
(Ответ для Grigory Ustinov на комментарий #5) > Кандидат готов начать процесс, выдайте, пожалуйста, сборочницу. *гитовницу. Но лучше сразу сборочницу. П.С. вообще говоря имеет смысл этот вопрос как-то обсудить, зачем выдавать человеку гитовницу, отдельно от сборочницы?
ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3.
Дальше!
ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. Адрес подписан на devel@. T/J/S -> 3.6.
Я совершенно забыл про этого товарища=( Прошу назначить ему рецензента поскорее. У меня уже очень давно нет замечаний к его деятельности.
Уже два месяца нет рецензента. Прошу призвать его.
Согласно новым правилам: https://www.altlinux.org/Team/Join/Review_Team с согласия других членов команды Review_Team беру кандидата на рецензирование. Прошу кандидата обращаться за аппрувом тасков сразу ко мне antohami@ (можно по почте или в matrix). По мере ревью ранее сделанных тасков буду отписываться сюда.
folder-color-switcher 1. Не надо делать такие зависимости: Requires: python3-module-caja Requires: /usr/bin/caja Во-первых, надо использовать макросы: Requires: %_bindir/caja Во-вторых, надо разбираться почему не нашлась зависимость на питоновский модуль. А не нашлось, потому что через Gobject Introspection используется. Чтобы зависимости находились, в спек нужно добавить: BuildRequires: rpm-build-gir 2. BuildRequires(pre): rpm-build-python3 В BuildRequires(pre) нужно добавлять только пакеты с макросами. Тут должно быть: BuildRequires: rpm-build-python3 3. Используйте всегда %prefix, а не вот так: %install mkdir -p %buildroot cp -a usr %buildroot/ Вместо этого можно так: mkdir -p %buildroot%prefix cp -a usr/* %buildroot%prefix К примеру во flatpak %prefix задаётся в /app. 4. Также просьба отделять %description пустой строкой для лучшей читаемости. А нет вот: Group: Graphical desktop/MATE %description common
pinta 1. Запакетили в обычный пакет: %_libdir/pkgconfig/%name.pc Так делать не надо. Это для devel. Скорее всего он никому никогда не понадобится, поэтому его следует просто не пакетить. Другой вариант - выделять devel пакет. 2. Packager: Vitaly Lipatov <lav@altlinux.ru> Последний раз в 15 году собирал. Уберите его из спека. И вообще, никогда этот тег в спеке не используйте.
musikcube Какова причина использования: %global __find_debuginfo_files %nil ?
(Ответ для Антон Мидюков на комментарий #13) > folder-color-switcher > > 1. Не надо делать такие зависимости: > > Requires: python3-module-caja > Requires: /usr/bin/caja > > Во-первых, надо использовать макросы: > Requires: %_bindir/caja > > Во-вторых, надо разбираться почему не нашлась зависимость на питоновский > модуль. > А не нашлось, потому что через Gobject Introspection используется. Чтобы > зависимости находились, в спек нужно добавить: > BuildRequires: rpm-build-gir Спасибо, буду знать, если с подобным столкнусь. Но здесь дело в другом: python3-module-caja не питоновский модуль, а так пакет назвали, по этому поводу бага есть: https://bugzilla.altlinux.org/50489 То есть тут указана зависимость на "caja-python" и следом на саму caja, так как без них плагин работать не будет, а "caja-python" по зависимостям не тянет caja. Ниже по второму плагину указана зависимость только на nemo-python, который вытягивает и nemo. Про использование макросов понятно. > > 2. BuildRequires(pre): rpm-build-python3 > > В BuildRequires(pre) нужно добавлять только пакеты с макросами. Тут должно > быть: > BuildRequires: rpm-build-python3 > > 3. Используйте всегда %prefix, а не вот так: > %install > mkdir -p %buildroot > cp -a usr %buildroot/ > > Вместо этого можно так: > mkdir -p %buildroot%prefix > cp -a usr/* %buildroot%prefix > > К примеру во flatpak %prefix задаётся в /app. > > 4. Также просьба отделять %description пустой строкой для лучшей читаемости. > А нет вот: > Group: Graphical desktop/MATE > %description common С учетом замечаний сделан таск: https://git.altlinux.org/tasks/419085/
(Ответ для Антон Мидюков на комментарий #14) > pinta > > 1. Запакетили в обычный пакет: > %_libdir/pkgconfig/%name.pc > > Так делать не надо. Это для devel. Скорее всего он никому никогда не > понадобится, поэтому его следует просто не пакетить. Другой вариант - > выделять devel пакет. > > 2. Packager: Vitaly Lipatov <lav@altlinux.ru> > > Последний раз в 15 году собирал. Уберите его из спека. И вообще, никогда > этот тег в спеке не используйте. С учетом замечаний собрал новую версию Pinta 3.1.2, уже на .NET 10, так как 9-ка находится в состоянии FTBFS: https://git.altlinux.org/tasks/419093/
(Ответ для Антон Мидюков на комментарий #15) > musikcube > > Какова причина использования: > %global __find_debuginfo_files %nil > ? Мне не понравилось, как собирался debug-info, в нем что-то не хватало. Поэтому я решил, что лучше без него. Возможно, причина кроется здесь https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob;f=script/post-build.sh;h=567502848ee836be51e09ef4a92663dab9eba149;hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l13 и здесь https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob;f=script/strip-nix.sh;h=e4ecea69e7406129b4cf7a7a7863bc55dd40ba46;hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l7 Апстрим принудительно делает strip бинарникам. Попробовать придумать патч я не решился.
Обновил плеер goodvibes до 0.8.4: https://git.altlinux.org/tasks/419089/
(Ответ для Alexander Kovalev на комментарий #18) > (Ответ для Антон Мидюков на комментарий #15) > > musikcube > > > > Какова причина использования: > > %global __find_debuginfo_files %nil > > ? > > Мне не понравилось, как собирался debug-info, в нем что-то не хватало. > Поэтому я решил, что лучше без него. Возможно, причина кроется здесь > https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob; > f=script/post-build.sh;h=567502848ee836be51e09ef4a92663dab9eba149; > hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l13 > и здесь > https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob; > f=script/strip-nix.sh;h=e4ecea69e7406129b4cf7a7a7863bc55dd40ba46; > hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l7 > > Апстрим принудительно делает strip бинарникам. Попробовать придумать патч я > не решился. Так задайте другое значение переменной, чтобы не стрипалось. Отключать debuginfo не нужно почти никогда.
(Ответ для Антон Мидюков на комментарий #20) > (Ответ для Alexander Kovalev на комментарий #18) > > (Ответ для Антон Мидюков на комментарий #15) > > > musikcube > > > > > > Какова причина использования: > > > %global __find_debuginfo_files %nil > > > ? > > > > Мне не понравилось, как собирался debug-info, в нем что-то не хватало. > > Поэтому я решил, что лучше без него. Возможно, причина кроется здесь > > https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob; > > f=script/post-build.sh;h=567502848ee836be51e09ef4a92663dab9eba149; > > hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l13 > > и здесь > > https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob; > > f=script/strip-nix.sh;h=e4ecea69e7406129b4cf7a7a7863bc55dd40ba46; > > hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l7 > > > > Апстрим принудительно делает strip бинарникам. Попробовать придумать патч я > > не решился. > > Так задайте другое значение переменной, чтобы не стрипалось. Отключать > debuginfo не нужно почти никогда. Разобрался с debuginfo, strip ни при чём. Ошибка была в использовании cmake вместо %cmake, поэтому потерялись %optflags. Кроме того, оказалось, что не хватает одного пакета в BuildRequires, и еще с помощью sed поправил пути в post-build скриптах, иначе часть команд не отрабатывалась, да и папка /lib/ в RPATH не нужна, так как не существует. Результат в таске: https://git.altlinux.org/tasks/419180/
(Ответ для Alexander Kovalev на комментарий #21) > (Ответ для Антон Мидюков на комментарий #20) > > (Ответ для Alexander Kovalev на комментарий #18) > > > (Ответ для Антон Мидюков на комментарий #15) > > > > musikcube > > > > > > > > Какова причина использования: > > > > %global __find_debuginfo_files %nil > > > > ? > > > > > > Мне не понравилось, как собирался debug-info, в нем что-то не хватало. > > > Поэтому я решил, что лучше без него. Возможно, причина кроется здесь > > > https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob; > > > f=script/post-build.sh;h=567502848ee836be51e09ef4a92663dab9eba149; > > > hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l13 > > > и здесь > > > https://git.altlinux.org/gears/m/musikcube.git?p=musikcube.git;a=blob; > > > f=script/strip-nix.sh;h=e4ecea69e7406129b4cf7a7a7863bc55dd40ba46; > > > hb=46460b526dbec76eca91bcfdbca5dc8423b8bc7d#l7 > > > > > > Апстрим принудительно делает strip бинарникам. Попробовать придумать патч я > > > не решился. > > > > Так задайте другое значение переменной, чтобы не стрипалось. Отключать > > debuginfo не нужно почти никогда. > > Разобрался с debuginfo, strip ни при чём. Ошибка была в использовании cmake > вместо %cmake, поэтому потерялись %optflags. Кроме того, оказалось, что не > хватает одного пакета в BuildRequires, и еще с помощью sed поправил пути в > post-build скриптах, иначе часть команд не отрабатывалась, да и папка /lib/ > в RPATH не нужна, так как не существует. Результат в таске: > > https://git.altlinux.org/tasks/419180/ Полагаться на sed такое себе. Лучше поддерживать патч.
Исправил на вариант с патчем. Патч сделал только для скрипта patch-rpath.sh, так как оказалось, что в скрипте strip-nix.sh на самом деле команды strip не работают, неверно написаны. А и нужно, чтобы не работали. https://git.altlinux.org/tasks/419180/
