Created attachment 17055 [details] SSH public key Псевдоним - darisishe Почта - konevsa@basealt.ru Ментор - Алексей Шабалин (shaba@altlinux.org) Сборка пакетов
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. В остальном хорошо. Поправил оба пакета, перезалил в те же таски
Таски проверил и одобрил. Считаю, что кандидат готов к самостоятельной сборке пакетов.
Пользователь добавлен в группу мейнтейнеров. Желаю удачного мейнтейнерства!
