Bug 39659

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@.
Пользователь добавлен в группу мантейнеров.

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