Detection of uuid_unparse_lower in #368 is broken

[Dagobert Michelsen]

The detection of uuid_unparse_lower in r12704 is broken, as the values of found are true and false and the simple if just checks if it is empty or not. This results in the test to always succeed and e.g. fail on Solaris which doesn't have uuid_unparse_lower. The attached patch fixes the issue.

[Dagobert Michelsen]

Fix detection of uuid_unparse_lower

[Olly Betts]

Hmm, I don't think I intended to write it that way, and it's worth changing I think, but it looks to me like it actually works currently, since if X runs command X and looks at its return code. Does that not work on Solaris for some reason?

$ for x in true false ; do echo "x=$x" ; if $x ; then echo yes ; fi ; done
x=true
yes
x=false

I'm wondering if the problem you're hitting is actually something else.

[Dagobert Michelsen]

Of course you are right, please discard my patch. The problem is as follows: the system has /usr/include/uuid/uuid.h without uuid_unparse_lower and we have an additional /opt/csw/include/uuid/uuid.h with uuid_unparse_lower which is detected. However, in the linking phase the -luuid is too early hence the /usr/lib/libuuid.so is pulled in missing the function resulting in the linkage error:

/bin/bash ./libtool  --tag=CXX   --mode=link /opt/SUNWspro/bin/CC -library=stlport4 -features=tmplife   -xO3 -m32 -xarch=sparc -lm   -lz -lm  -luuid -lm  -lnsl  ...

Maybe the exact location of libuuid.so could be specified during configure-time?

[Olly Betts]

Can you attach config.log from this build?

[Dagobert Michelsen]

config.log

[Dagobert Michelsen]

Sure, just drop me a note if you need anything else. For now I worked around the issue by adding /opt/csw/lib to CXXFLAGS being before -luuid.

[Olly Betts]

So what's happening is that -luuid gets added to XAPIAN_LDFLAGS in configure, which ends up on the link line before $LDFLAGS from the environment. The problem is that -l options don't belong in *LDFLAGS, though I'm not sure how easy this case is going to be to untangle. Still, we should try before 1.3.2.

[Olly Betts]

Fixed in trunk r17578. We should probably backport this to 1.2, but it would be good to get confirmation that this definitely fixes the build for you.

[Olly Betts]

And I've simplified the test for uuid_unparse_lower() to avoid if $found in r17579.

[Olly Betts]

Backported for 1.2.16 in r17585 and r17586.