Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1282 bytes --]

On Mon, Feb 19, 2007 at 01:38:02PM +0300, Upravdom Team Robot wrote:
> Dear Igor Zubkov (ALTLinux Team)!
> The libasyncns-0.2-alt1 package has been rejected from incoming for
> x86_64 architecture.
> 
> Possible reason follows:
> checking for working fork... yes
> checking for working vfork... (cached) yes
> checking for stdlib.h... (cached) yes
> checking for GNU libc compatible malloc... yes
> checking sys/select.h usability... yes
> checking sys/select.h presence... yes
> checking for sys/select.h... yes
> checking for sys/socket.h... (cached) yes
> checking types of arguments for select... int,fd_set *,struct timeval *
> checking return type of signal handlers... void
> checking for memset... yes
> checking for select... yes
> checking for strndup... yes
> checking for setresuid... yes
> checking for setreuid... yes
> checking for strerror... yes
> checking for gethostbyname in -lnsl... yes
> checking for connect in -lsocket... no
> checking for res_query... no
> checking for res_query in -lresolv... no
> checking for res_query in -lbind... no
> configure: error: res_query not found
> error: Bad exit status from /usr/src/tmp/rpm-tmp.16815 (%build)

Это неправильная проверка, имя символа не обязано быть res_query.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Igor Zubkov]

В сообщении от Monday 19 February 2007 18:36:36 Dmitry V. Levin написал(а):
> > checking for res_query... no
> > checking for res_query in -lresolv... no
> > checking for res_query in -lbind... no
> > configure: error: res_query not found
> > error: Bad exit status from /usr/src/tmp/rpm-tmp.16815 (%build)
>
> Это неправильная проверка, имя символа не обязано быть res_query.

Мммм... А какая должна быть "правильной"?

-- 
icesik

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Igor Zubkov]

В сообщении от Monday 19 February 2007 18:36:36 Dmitry V. Levin написал(а):
> > checking for res_query... no
> > checking for res_query in -lresolv... no
> > checking for res_query in -lbind... no
> > configure: error: res_query not found
> > error: Bad exit status from /usr/src/tmp/rpm-tmp.16815 (%build)
>
> Это неправильная проверка, имя символа не обязано быть res_query.

А каким оно должно быть?

-- 
icesik

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 569 bytes --]

On Tue, Feb 20, 2007 at 06:05:32PM +0200, Igor Zubkov wrote:
> В сообщении от Monday 19 February 2007 18:36:36 Dmitry V. Levin написал(а):
> > > checking for res_query... no
> > > checking for res_query in -lresolv... no
> > > checking for res_query in -lbind... no
> > > configure: error: res_query not found
> > > error: Bad exit status from /usr/src/tmp/rpm-tmp.16815 (%build)
> >
> > Это неправильная проверка, имя символа не обязано быть res_query.
> 
> А каким оно должно быть?

__res_query

$ subst -p 's/res_query/__&/g' configure*


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Igor Zubkov]

В сообщении от Tuesday 20 February 2007 18:21:33 Dmitry V. Levin написал(а):
> On Tue, Feb 20, 2007 at 06:05:32PM +0200, Igor Zubkov wrote:
> > В сообщении от Monday 19 February 2007 18:36:36 Dmitry V. Levin 
написал(а):
> > > > checking for res_query... no
> > > > checking for res_query in -lresolv... no
> > > > checking for res_query in -lbind... no
> > > > configure: error: res_query not found
> > > > error: Bad exit status from /usr/src/tmp/rpm-tmp.16815 (%build)
> > >
> > > Это неправильная проверка, имя символа не обязано быть res_query.
> >
> > А каким оно должно быть?
>
> __res_query
>
> $ subst -p 's/res_query/__&/g' configure*

Странно, но с этим у меня на i586 собралось. Ещё бы обьяснение бы внятное для 
автора и патч я ему отошлю (я с ним иногда переписываюсь, и все мои патчи он 
принимал).

-- 
icesik

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1389 bytes --]

On Tue, Feb 20, 2007 at 06:52:31PM +0200, Igor Zubkov wrote:
> В сообщении от Tuesday 20 February 2007 18:21:33 Dmitry V. Levin написал(а):
> > On Tue, Feb 20, 2007 at 06:05:32PM +0200, Igor Zubkov wrote:
> > > В сообщении от Monday 19 February 2007 18:36:36 Dmitry V. Levin 
> написал(а):
> > > > > checking for res_query... no
> > > > > checking for res_query in -lresolv... no
> > > > > checking for res_query in -lbind... no
> > > > > configure: error: res_query not found
> > > > > error: Bad exit status from /usr/src/tmp/rpm-tmp.16815 (%build)
> > > >
> > > > Это неправильная проверка, имя символа не обязано быть res_query.
> > >
> > > А каким оно должно быть?
> >
> > __res_query
> >
> > $ subst -p 's/res_query/__&/g' configure*
> 
> Странно, но с этим у меня на i586 собралось.

Потому что на i586 в glibc есть алиас res_query -> __res_query (для
обратной совместимости).  На более современных архитектурах (в т.ч.
x86-64) этого алиаса нет.

> Ещё бы обьяснение бы внятное для 
> автора и патч я ему отошлю
> (я с ним иногда переписываюсь, и все мои патчи он принимал).

Если пакет поддерживает архитектуры без glibc, то просто заменить
res_query на __res_query нельзя.

Переносимым исправлением будет изменённая проверка символов
(res_query или __res_query).

Не исключено, что для res_* уже существует специальный m4-макрос.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Igor Zubkov]

[-- Attachment #1: Type: text/plain, Size: 1098 bytes --]

В сообщении от Tuesday 20 February 2007 19:04:17 Dmitry V. Levin написал(а):
> On Tue, Feb 20, 2007 at 06:52:31PM +0200, Igor Zubkov wrote:
> > В сообщении от Tuesday 20 February 2007 18:21:33 Dmitry V. Levin 
написал(а):
> > > $ subst -p 's/res_query/__&/g' configure*
> >
> > Странно, но с этим у меня на i586 собралось.
>
> Потому что на i586 в glibc есть алиас res_query -> __res_query (для
> обратной совместимости).  На более современных архитектурах (в т.ч.
> x86-64) этого алиаса нет.
>
> > Ещё бы обьяснение бы внятное для
> > автора и патч я ему отошлю
> > (я с ним иногда переписываюсь, и все мои патчи он принимал).
>
> Если пакет поддерживает архитектуры без glibc, то просто заменить
> res_query на __res_query нельзя.
>
> Переносимым исправлением будет изменённая проверка символов
> (res_query или __res_query).
>
> Не исключено, что для res_* уже существует специальный m4-макрос.

Ага. Спасиюо за обьяснение. Как оказалось, автор в курсе и у него уже есть 
патч от пиплов из debian. Он в аттаче. Только я не поминаю что он делает. В 
общем, прошу помощи с пониманием.

-- 
icesik

[-- Attachment #2: resolv-configure-check.patch --]
[-- Type: text/x-diff, Size: 787 bytes --]

Index: configure.ac
===================================================================
--- configure.ac	(revision 30)
+++ configure.ac	(working copy)
@@ -96,8 +96,16 @@
 
 AC_CHECK_FUNC(res_query, , 
   [AC_CHECK_LIB(resolv, res_query, [], 
-    [ AC_CHECK_LIB(bind, res_query, 
-                   [], [ AC_MSG_ERROR(res_query not found) ] ) ] )
+    [ save_libs="$LIBS"
+      LIBS="-lresolv $LIBS"
+      AC_MSG_CHECKING([for res_query in -lresolv (alternate version)])
+      AC_LINK_IFELSE(
+        [AC_LANG_PROGRAM([[#include <resolv.h>]], [[res_query(0,0,0,0,0)]])], 
+        [ AC_MSG_RESULT(yes) ],  
+        [ AC_MSG_RESULT(no)
+          LIBS="$save_libs"
+          AC_CHECK_LIB(bind, res_query, 
+            [], [ AC_MSG_ERROR(res_query not found) ] ) ] ) ] )
   ]
 )
 

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Damir Shayhutdinov]

> Ага. Спасиюо за обьяснение. Как оказалось, автор в курсе и у него уже есть
> патч от пиплов из debian. Он в аттаче. Только я не поминаю что он делает. В
> общем, прошу помощи с пониманием.
Заменяет проверку присутствия символа в библиотеке на компиляцию и
лигковку минипрограммки, которая включают нужный header и вызывает
нужную функцию.

Поскольку тут включается нужный header <resolv.h> - то это будет
работать независимо от того, является ли res_query() макросом,
функцией или еще чем-нибудь.

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 653 bytes --]

On Wed, Feb 21, 2007 at 04:17:07PM +0300, Damir Shayhutdinov wrote:
> > Ага. Спасиюо за обьяснение. Как оказалось, автор в курсе и у него уже есть
> > патч от пиплов из debian. Он в аттаче. Только я не поминаю что он делает. В
> > общем, прошу помощи с пониманием.
> Заменяет проверку присутствия символа в библиотеке на компиляцию и
> лигковку минипрограммки, которая включают нужный header и вызывает
> нужную функцию.
> 
> Поскольку тут включается нужный header <resolv.h> - то это будет
> работать независимо от того, является ли res_query() макросом,
> функцией или еще чем-нибудь.

Вроде нормальный патч, прикладывайте.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Igor Zubkov]

В сообщении от Wednesday 21 February 2007 15:22:48 Dmitry V. Levin написал(а):
> > Заменяет проверку присутствия символа в библиотеке на компиляцию и
> > лигковку минипрограммки, которая включают нужный header и вызывает
> > нужную функцию.
> >
> > Поскольку тут включается нужный header <resolv.h> - то это будет
> > работать независимо от того, является ли res_query() макросом,
> > функцией или еще чем-нибудь.
>
> Вроде нормальный патч, прикладывайте.

Сейчас уйдёт на сборку.

-- 
icesik

Re: [devel] [Sisyphus-incominger] E: incoming x86_64 reject: libasyncns-0.2-alt1

[Igor Zubkov]

В сообщении от Tuesday 20 February 2007 18:21:33 Dmitry V. Levin написал(а):
> > А каким оно должно быть?
>
> __res_query
>
> $ subst -p 's/res_query/__&/g' configure*

libasyncns-0.2-alt2 с этим фиксом улетел на сборку.

-- 
icesik