Псевдоним: egori Почта: egori@basealt.ru Ментор: Алексей Никифоров <darktemplar@basealt.ru> Цель: Сборка пакетов, участие в поддержке и разработке ALT Linux.
Created attachment 9183 [details] Public ssh key
Created attachment 9184 [details] Public gpg key
Created attachment 9185 [details] Public gpg key (fixed)
(Ответ для Egor Ignatov на комментарий #0) > Ментор: Алексей Никифоров <darktemplar@basealt.ru> Подтверждаю. Прошу предоставить доступ на git.altlinux.org.
(Ответ для Egor Ignatov на комментарий #0) > Почта: egori@basealt.ru (Ответ для Egor Ignatov на комментарий #1) > Создано вложение 9183 [details] > Public ssh key (Ответ для Egor Ignatov на комментарий #3) > Создано вложение 9185 [details] > Public gpg key (fixed) (Ответ для Aleksei Nikiforov на комментарий #4) > (Ответ для Egor Ignatov на комментарий #0) > > Ментор: Алексей Никифоров <darktemplar@basealt.ru> > Подтверждаю. Ok.
ssh ключ на gitery.alt зарегистрирован. ssh ключ на gyle.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.4.
Считаю, что кандидат готов к следующему шагу. Прошу предоставить доступ на сборочницу. Текущий репозиторий: http://git.altlinux.org/people/egori/packages/?p=kitty.git;a=summary
Пакет alt-gpgkeys обновлён. T/J/S -> 3.4.
Считаю, что кандидат готов.
Что-то зависло, нужно что бы кто-то произнёс заклинаение призыва ещё одного ментора.
Призван ещё один человек (bircoph@) для независимой оценки готовности кандидата.
Егор, я обнаружил в Вашем пакете kitty следующую проблему: 056-debuginfo.brp: WARNING: debuginfo without debug sources. Пожалуйста, разберитесь в том, почему это происходит и по возможности исправьте.
Ещё я посмотрел Ваш репозиторий audit, т.к. я работал с этим пакетом при его адаптации для Эльбруса и свежих ядер на нём и он мне не безразличен. Меня озадачил Ваш следующий коммит: commit 2b74fdf058f4f13fc9c6d0a0c9879bd0ac92025b Author: Egor Ignatov <egori@altlinux.org> Date: Fri Feb 19 12:32:39 2021 +0300 Fix configure.ac to not add libpthread to LIBS diff --git a/configure.ac b/configure.ac index f2f2950..1ba3276 100644 --- a/configure.ac +++ b/configure.ac @@ -86,8 +86,10 @@ AC_LINK_IFELSE( [] ) dnl; pthread_yield is used in zos-remote +OLDLIBS="$LIBS" AC_SEARCH_LIBS(pthread_yield, pthread, [AC_DEFINE(HAVE_PTHREAD_YIELD, 1, [Define to 1 if we have pthread_yield])], []) +LIBS="$OLDLIBS" ALLWARNS="" ALLDEBUG="-g" Вы действительно считаете такое решение нормальной практикой? Я бы хотел видеть поменьше подобных костылей в репозитории, особенно для важных пакетов. Предлагаю или полностью удалить проверку (с учётом возможной цепной реакции), или, ещё лучше, добавить флаг --enable-pthreads для управления данной фукнциональностью из spec. (Вопрос о необходимости audit с тредами или без я осталвляю за рамками данной дискуссии, т.к. это отдельный вопрос.)
(In reply to Andrew Savchenko from comment #13) > Ещё я посмотрел Ваш репозиторий audit, т.к. я работал с этим пакетом при его > адаптации для Эльбруса и свежих ядер на нём и он мне не безразличен. > > Меня озадачил Ваш следующий коммит: > > commit 2b74fdf058f4f13fc9c6d0a0c9879bd0ac92025b > Author: Egor Ignatov <egori@altlinux.org> > Date: Fri Feb 19 12:32:39 2021 +0300 > > Fix configure.ac to not add libpthread to LIBS > > diff --git a/configure.ac b/configure.ac > index f2f2950..1ba3276 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -86,8 +86,10 @@ AC_LINK_IFELSE( > [] > ) > dnl; pthread_yield is used in zos-remote > +OLDLIBS="$LIBS" > AC_SEARCH_LIBS(pthread_yield, pthread, > [AC_DEFINE(HAVE_PTHREAD_YIELD, 1, [Define to 1 if we have > pthread_yield])], []) > +LIBS="$OLDLIBS" > > ALLWARNS="" > ALLDEBUG="-g" > > Вы действительно считаете такое решение нормальной практикой? Я бы хотел > видеть поменьше подобных костылей в репозитории, особенно для важных пакетов. Мне кажется, что это фикс той проблемы, о которой идёт речь в https://lists.altlinux.org/pipermail/devel/2021-February/213703.html $ git grep -F HAVE_PTHREAD_YIELD audisp/plugins/zos-remote/zos-remote-queue.c:#ifndef HAVE_PTHREAD_YIELD audisp/plugins/zos-remote/zos-remote-queue.c:#ifdef HAVE_PTHREAD_YIELD configure.ac: [AC_DEFINE(HAVE_PTHREAD_YIELD, 1, [Define to 1 if we have pthread_yield])], []) Этот zos-remote-queue уже использует другие функции из libthread и линкуется с -lpthread напрямую. Т.е. этот фикс корректный, хотя это и неочевидно без описания и контекста. А как бы ты это пофиксил более аккуратно?
Нормальный фикс, и upstream тоже с этим согласен: https://github.com/linux-audit/audit-userspace/commit/92db0dca245191ac2f14e33efbf2309d576f532a
(In reply to Andrew Savchenko from comment #12) > Егор, я обнаружил в Вашем пакете kitty следующую проблему: > > 056-debuginfo.brp: WARNING: debuginfo without debug sources. > > Пожалуйста, разберитесь в том, почему это происходит и по возможности > исправьте. Андрей, спасибо, что обратили мое внимание на эту ошибку, исправил: http://git.altlinux.org/gears/k/kitty.git
(In reply to Dmitry V. Levin from comment #14) > (In reply to Andrew Savchenko from comment #13) > Мне кажется, что это фикс той проблемы, о которой идёт речь в > https://lists.altlinux.org/pipermail/devel/2021-February/213703.html > > $ git grep -F HAVE_PTHREAD_YIELD > audisp/plugins/zos-remote/zos-remote-queue.c:#ifndef HAVE_PTHREAD_YIELD > audisp/plugins/zos-remote/zos-remote-queue.c:#ifdef HAVE_PTHREAD_YIELD > configure.ac: [AC_DEFINE(HAVE_PTHREAD_YIELD, 1, [Define to 1 if we have > pthread_yield])], []) > > Этот zos-remote-queue уже использует другие функции из libthread и линкуется > с -lpthread напрямую. Согласен, в полном контексте патч выглядит уместнее. В идеале хорошо бы подобные коммиты подробнее комментировать, ну хоть ну же ссылку на ML дать. > Т.е. этот фикс корректный, хотя это и неочевидно без описания и контекста. > А как бы ты это пофиксил более аккуратно? Я бы использовал AC_CHECK_HEADER. Но раз уж апстрим принял такое же изменение на базе обхода с OLDLIBS, то не вижу смысла делать иначе, т.к. в данном случае уход от апстрима будет бóльшим злом.
(In reply to Egor Ignatov from comment #16) > (In reply to Andrew Savchenko from comment #12) > > Егор, я обнаружил в Вашем пакете kitty следующую проблему: > > > > 056-debuginfo.brp: WARNING: debuginfo without debug sources. > > > > Пожалуйста, разберитесь в том, почему это происходит и по возможности > > исправьте. > > Андрей, спасибо, что обратили мое внимание на эту ошибку, исправил: > http://git.altlinux.org/gears/k/kitty.git Спасибо. С исправлением согласен. Подтверждаю готовность кандидата к самостоятельной работе в Сизифе.
(In reply to Andrew Savchenko from comment #18) > Подтверждаю готовность кандидата к самостоятельной работе в Сизифе.
Адрес подписан на список рассылки devel@. Пользователь добавлен в группу мантейнеров. Желаю удачного мантейнерства!
