From: Piotr Balcer <[email protected]>

Compilation on Ubuntu 18.04 fails with:

  /usr/bin/ld: util/keys.o: undefined reference to symbol 
'keyctl_read_alloc@@KEYUTILS_0.3'
  /lib/x86_64-linux-gnu/libkeyutils.so.1: error adding symbols: DSO missing 
from command line

Seems like libkeyutils is incorrectly linked against libndctl,
where in reality it's the ndctl application that uses keyutils.

Cc: Dave Jiang <[email protected]>
Fixes: 86b078b44275 ("ndctl: add passphrase management commands")
Signed-off-by: Piotr Balcer <[email protected]>
Signed-off-by: Vishal Verma <[email protected]>
---
 ndctl/Makefile.am     | 4 ++++
 ndctl/lib/Makefile.am | 4 ----
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/ndctl/Makefile.am b/ndctl/Makefile.am
index 3287dbb..502271e 100644
--- a/ndctl/Makefile.am
+++ b/ndctl/Makefile.am
@@ -47,6 +47,10 @@ ndctl_LDADD =\
        $(KMOD_LIBS) \
        $(JSON_LIBS)
 
+if ENABLE_KEYUTILS
+ndctl_LDADD += -lkeyutils
+endif
+
 if ENABLE_TEST
 ndctl_SOURCES += ../test/libndctl.c \
                 ../test/dsm-fail.c \
diff --git a/ndctl/lib/Makefile.am b/ndctl/lib/Makefile.am
index 99eaae0..7797039 100644
--- a/ndctl/lib/Makefile.am
+++ b/ndctl/lib/Makefile.am
@@ -30,10 +30,6 @@ libndctl_la_LIBADD =\
        $(UUID_LIBS) \
        $(KMOD_LIBS)
 
-if ENABLE_KEYUTILS
-libndctl_la_LIBADD += -lkeyutils
-endif
-
 EXTRA_DIST += libndctl.sym
 
 libndctl_la_LDFLAGS = $(AM_LDFLAGS) \
-- 
2.20.1

_______________________________________________
Linux-nvdimm mailing list
[email protected]
https://lists.01.org/mailman/listinfo/linux-nvdimm

Reply via email to