On 08/12/2018 11:46 PM, Julio Faracco wrote:
Hi guys,
Em sex, 10 de ago de 2018 às 11:23, Jim Fehlig <jfehlig(a)suse.com> escreveu:
>
> On 08/09/2018 11:45 PM, Michal Privoznik wrote:
>> There are few places where dlopen() is called. This call means we
>> have to link with DLOPEN_LIBS. However, instead of having each
>> final, installable library linking with it, move the directive to
>> the source that introduced the dependency.
>>
>> Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
>
> Reviewed-by: Jim Fehlig <jfehlig(a)suse.com>
>
>> ---
>> src/Makefile.am | 2 ++
>> src/util/Makefile.inc.am | 1 +
>> tools/Makefile.am | 4 ++--
>> 3 files changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/Makefile.am b/src/Makefile.am
>> index a4f213480e..61876cf382 100644
>> --- a/src/Makefile.am
>> +++ b/src/Makefile.am
>> @@ -752,6 +752,7 @@ libvirt_setuid_rpc_client_la_CFLAGS = \
>> $(AM_CFLAGS) \
>> $(SECDRIVER_CFLAGS) \
>> $(XDR_CFLAGS) \
>> + $(DLOPEN_LIBS) \
>> $(NULL)
>> endif WITH_SETUID_RPC_CLIENT
I think that DLOPEN_LIBS is not needed for setuid_rpc_client.
See the error when I enable RPC libraries:
clang: error: -ldl: 'linker' input unused
[-Werror,-Wunused-command-line-argument]
Makefile:9014: recipe for target
'util/libvirt_setuid_rpc_client_la-viralloc.lo' failed
Ah, so the library compiles virjsoncompact.c in which I used as an
indication where to put DLOPEN_LIBS. But now that I look at it closer it
doesn't make sense for the lib to compile in the file because in
config-post.h we clearly disable JANSSON support.
Sigh, this jansson has brought more problems than it solved.
Anyway, I think this is the fix:
diff --git i/src/Makefile.am w/src/Makefile.am
index 61876cf382..d9bce1e8ab 100644
--- i/src/Makefile.am
+++ w/src/Makefile.am
@@ -689,7 +689,6 @@ libvirt_setuid_rpc_client_la_SOURCES = \
util/virhashcode.c \
util/virhostcpu.c \
util/virjson.c \
- util/virjsoncompat.c \
util/virlog.c \
util/virobject.c \
util/virpidfile.c \
@@ -752,7 +751,6 @@ libvirt_setuid_rpc_client_la_CFLAGS = \
$(AM_CFLAGS) \
$(SECDRIVER_CFLAGS) \
$(XDR_CFLAGS) \
- $(DLOPEN_LIBS) \
$(NULL)
endif WITH_SETUID_RPC_CLIENT
Can you test it please? I'm no longer confident around jansson.
Michal