[SSSD] [PATCH] Check our libraries properly resolve symbols

Simo Sorce simo at redhat.com
Sat Sep 7 17:17:12 UTC 2013


On Sat, 2013-09-07 at 12:39 -0400, Simo Sorce wrote:
> On Sat, 2013-09-07 at 11:42 +0200, Lukas Slebodnik wrote:
> > >+libdlopen_test_providers_la_LDFLAGS = \
> > >+    -module \
> > >+    -avoid-version \
> > >+    -Wl,--version-script,$(srcdir)/src/providers/sssd_be.exports \
> > add "-rpath $(abs_builddir) \"
> > libdlopen_test_providers will be dynamic library with this line.
> > 
> I had already tried this trick and it did not work, but the exact line I
> used was slightly different.

Ah it did work with this syntax, I had tried with -Wl,-rpath previously.
Thanks a lot.

> I'll retry and if it works I'll send an updated patch I do preferer
> check_LTLIBRARY as well, rather than the current hack.

so now check_LTLIBRARY is used instead, moreover I added guards around
libraries that we compiled only conditionally, I had to add a couple of
statements in m4 files to make stuff appear in config.in.
Also I added the 2 missing python bindings.

This should be final, it all works here.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-tests-Add-dlopen-test-to-make-sure-modules-works.patch
Type: text/x-patch
Size: 8149 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130907/408133f3/attachment.bin>


More information about the sssd-devel mailing list