diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2013-04-19 11:27:59 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-04-22 15:35:58 +0200 |
| commit | 661a983c73520a59803729108812396323c8366b (patch) | |
| tree | f9161c2d07850c062dcbf5facdd9fcf828c3ac8b | |
| parent | f4fddaf1bad5f82920ae3ec2af90b42309472a89 (diff) | |
| download | sssd-661a983c73520a59803729108812396323c8366b.tar.gz sssd-661a983c73520a59803729108812396323c8366b.tar.xz sssd-661a983c73520a59803729108812396323c8366b.zip | |
tests: Link the simple access tests with -ldl
In SSSD, we use dlopen() and dlsym() in two files
src/providers/data_provider_be.c and src/providers/proxy/proxy_init.c.
Hence we should explicitly link with -ldl also in simple_access-tests.
SSSD can be compiled with two crypto libraries nss or libcrypto. NSS has
dependency nspr which depends on "libdl and libpthread" This is a reason why
compilation of test did not fail even if -ldl was not explicitly added to
simple_access_tests_LDADD. But libcrypto doesn't depend on libdl, so in
this case compilation of tests will not be successful.
Upstream nspr 4.9 has two ways have to obtain metainformation about
libraries
pkg-config and own script nspr-config. First one doesn't list "-ldl"
"-lpthread" but second one lists both "-ldl" "-lpthread"
That's also why the Ubuntu maintainer found this bug -- Fedora has got
patched version of nspr, but Debian (Ubuntu) doesn't
| -rw-r--r-- | Makefile.am | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Makefile.am b/Makefile.am index 0ccb747b7..f0ee88b3d 100644 --- a/Makefile.am +++ b/Makefile.am @@ -1019,6 +1019,7 @@ simple_access_tests_CFLAGS = \ $(CHECK_CFLAGS) \ -DUNIT_TESTING simple_access_tests_LDADD = \ + -ldl \ $(SSSD_LIBS) \ $(CARES_LIBS) \ $(CHECK_LIBS) \ |