r23186 - gnucash/trunk - Bug 654196 - "make check" fails when built with --enable-dbi

[John Ralls]

On Sep 24, 2013, at 1:43 PM, Christian Stimming <christian at cstimming.de> wrote:

> Am Sonntag, 22. September 2013, 14:56:52 schrieb John Ralls:
>>>>> The new test fails for me as well: Kubuntu 11.10, and config.log
>>>>> says:
>>>>> 
>>>>> configure:20323: checking Looking for at least one supported DBD
>>>>> module
>>>>> configure:20343: gcc -o conftest -I/home/cs/usr/include   -ldl
>>>>> conftest.c - lpthread
>>>>>> &5
>>>>> /tmp/ccISRVMn.o: In function `main':
>>>>> conftest.c:(.text+0xf): undefined reference to `dlopen'
>> 
>> So libtool.m4's call of AC_CHECK_LIB([ldl], [dlopen],...) found dlopen in
>> libdl.so.
>> 
>> More likely then that the command  gcc -o conftest -I/home/cs/usr/include  
>> -ldl conftest.c - lpthread isn't trying to link libdl. Which is strange,
>> especially since it works fine on Debian Wheezy 64.
>> 
>> Hmm. That space between - and lpthread is bothersome.  Try this patch and
>> see if it changes anything: 
> 
> Indeed, with this patch it works. The log says
> 
> configure:20283: checking for dbi/dbi.h
> configure:20283: result: yes
> configure:20320: checking Looking for at least one supported DBD module
> configure:20340: gcc -o conftest -I/home/cs/usr/include   conftest.c -ldl -
> lpthread  >&5
> configure:20340: $? = 0
> configure:20340: ./conftest
> configure:20340: $? = 0
> 
> Does this mean it "just" was the -ldl had to come after the conftest.c 
> argument? Seems a bit weird to me that it works for you, though.
> 
> In any case, I'd be glad if you could commit this patch. Thanks!

Yay!

Yes, it probably does, though it's weird that only your compiler seemed to mind.

Done, r23195.

Regards,
John Ralls