Bug 51830

Created attachment 17056 [details]
GPG public key

Принимаю кандидата.

Ментор есть, ключи в порядке.
T/J/S -> 1.3.

Кандидат готов к вступлению, прошу
* Создать email alias для кандидата
* Зарегистрировать SSH-ключ кандидата в gitery.alt.

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

T/J/S -> 2.3.

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

(In reply to Alexey Shabalin from comment #6)
> готов собирать паркеты. Прошу предоставить доступ к сборочнице.
А ламинат? :)

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

T/J/S -> 3.6.

Кандидат готов собирать пакеты в сизиф.
Прошу призвать рецензента.

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

T/J/S -> 4.2.

По пакету libyui вопрос к кандидату:
Почему используете в макросах {} без необходимости? В Альте так не принято.
Вызывает подозрение, что спек откуда-то стянут без переосмысления.

Также к сведению, что вместо
Requires: зависимость >= %{version}-%{release}
лучше указывать:
Requires: зависимость >= %EVR
то есть с учётом ещё и Epoch пакета.

В конце спека три пустых строки, когда достаточно всего одной.

Пакет blivet был переведён кандидатом с srpm на gear, но собрано всё одним коммитом:
https://git.altlinux.org/gears/b/blivet.git?p=blivet.git;a=shortlog;h=6da6338767b713ea4083bfcd4a9e79e0ab7fb9be

@darisishe, так делать ни в коем случае нельзя. Вы должны были или продолжать собирать srpm:
https://www.altlinux.org/Gear/Обновление_SRPM

или разделять свои действия на коммиты (обновление исходников один коммит, а правка спека другой).
Жаль, что ментор вам этого не объяснил.

(Ответ для Антон Мидюков на комментарий #10)
> По пакету libyui вопрос к кандидату:
> Почему используете в макросах {} без необходимости? В Альте так не принято.
> Вызывает подозрение, что спек откуда-то стянут без переосмысления.

За основу брал апстримный спек, но там нужно было много перелопатить, так что переосмысление было :)
По поводу скобок учту на будущее

> Также к сведению, что вместо
> Requires: зависимость >= %{version}-%{release}
> лучше указывать:
> Requires: зависимость >= %EVR
> то есть с учётом ещё и Epoch пакета.
> 
> В конце спека три пустых строки, когда достаточно всего одной.

https://git.altlinux.org/tasks/392852/
Поправил спек, заодно обновил пакет.

(Ответ для Антон Мидюков на комментарий #11)
> Пакет blivet был переведён кандидатом с srpm на gear, но собрано всё одним
> коммитом:
> https://git.altlinux.org/gears/b/blivet.git?p=blivet.git;a=shortlog;
> h=6da6338767b713ea4083bfcd4a9e79e0ab7fb9be
> 
> @darisishe, так делать ни в коем случае нельзя. Вы должны были или
> продолжать собирать srpm:
> https://www.altlinux.org/Gear/Обновление_SRPM
> 
> или разделять свои действия на коммиты (обновление исходников один коммит, а
> правка спека другой).
> Жаль, что ментор вам этого не объяснил.

Переводил на gear, т.к. возникла необходимость собирать с мастер-ветки (апстрим редко выставляет теги с тарболлами, но при этом было достаточно много коммитов с важными фиксами).
Поправил, сделал нормальный gear-репозиторий. Т.к. апстрим хранит переводы в отдельном подмодуле, пришлось их подцеплять через merge с `-s ours`.

https://git.altlinux.org/tasks/393206/

(Ответ для darisishe@altlinux.org на комментарий #12)
> (Ответ для Антон Мидюков на комментарий #10)
> > По пакету libyui вопрос к кандидату:
> > Почему используете в макросах {} без необходимости? В Альте так не принято.
> > Вызывает подозрение, что спек откуда-то стянут без переосмысления.
> 
> За основу брал апстримный спек, но там нужно было много перелопатить, так
> что переосмысление было :)
> По поводу скобок учту на будущее
> 
> > Также к сведению, что вместо
> > Requires: зависимость >= %{version}-%{release}
> > лучше указывать:
> > Requires: зависимость >= %EVR
> > то есть с учётом ещё и Epoch пакета.
> > 
> > В конце спека три пустых строки, когда достаточно всего одной.
> 
> https://git.altlinux.org/tasks/392852/
> Поправил спек, заодно обновил пакет.

%_libdir/yui/%name-qt.so.%{so_version}*

Надёжнее двумя строками сделать:
%_libdir/yui/%name-qt.so.%so_version
%_libdir/yui/%name-qt.so.%so_version.*

(Ответ для darisishe@altlinux.org на комментарий #13)
> (Ответ для Антон Мидюков на комментарий #11)
> > Пакет blivet был переведён кандидатом с srpm на gear, но собрано всё одним
> > коммитом:
> > https://git.altlinux.org/gears/b/blivet.git?p=blivet.git;a=shortlog;
> > h=6da6338767b713ea4083bfcd4a9e79e0ab7fb9be
> > 
> > @darisishe, так делать ни в коем случае нельзя. Вы должны были или
> > продолжать собирать srpm:
> > https://www.altlinux.org/Gear/Обновление_SRPM
> > 
> > или разделять свои действия на коммиты (обновление исходников один коммит, а
> > правка спека другой).
> > Жаль, что ментор вам этого не объяснил.
> 
> Переводил на gear, т.к. возникла необходимость собирать с мастер-ветки
> (апстрим редко выставляет теги с тарболлами, но при этом было достаточно
> много коммитов с важными фиксами).
> Поправил, сделал нормальный gear-репозиторий. Т.к. апстрим хранит переводы в
> отдельном подмодуле, пришлось их подцеплять через merge с `-s ours`.
> 
> https://git.altlinux.org/tasks/393206/

В спеке  Source1 распаковывается гораздо проще:

diff --git a/.gear/blivet.spec b/.gear/blivet.spec
index 3707390f..60b34a14 100644
--- a/.gear/blivet.spec
+++ b/.gear/blivet.spec
@@ -86,9 +86,7 @@ The python3-%name is a python3 package for examining and modifying storage
 configuration.
 
 %prep
-%setup
-mkdir -p po/
-tar -xf %SOURCE1 -C po --strip-components 1
+%setup -a1
 
 %build
 make PYTHON=%__python3

И заодно поправьте License в формат SPDX. В остальном хорошо.

(Ответ для Антон Мидюков на комментарий #14)
> 
> %_libdir/yui/%name-qt.so.%{so_version}*
> 
> Надёжнее двумя строками сделать:
> %_libdir/yui/%name-qt.so.%so_version
> %_libdir/yui/%name-qt.so.%so_version.*

(Ответ для Антон Мидюков на комментарий #15)
> 
> В спеке  Source1 распаковывается гораздо проще:
> 
> diff --git a/.gear/blivet.spec b/.gear/blivet.spec
> index 3707390f..60b34a14 100644
> --- a/.gear/blivet.spec
> +++ b/.gear/blivet.spec
> @@ -86,9 +86,7 @@ The python3-%name is a python3 package for examining and
> modifying storage
>  configuration.
>  
>  %prep
> -%setup
> -mkdir -p po/
> -tar -xf %SOURCE1 -C po --strip-components 1
> +%setup -a1
>  
>  %build
>  make PYTHON=%__python3
> 
> И заодно поправьте License в формат SPDX. В остальном хорошо.

Поправил оба пакета, перезалил в те же таски

Таски проверил и одобрил.

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

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

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