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/

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

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

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

Мои поздравления!

Большое спасибо всем!