11:08 a.m.
Yes, I am running this on ubuntu.
And I will sent it as a formal fix patch later.
-----Original Message-----
From: Dan Williams <dan.j.williams(a)intel.com>
Sent: Friday, December 27, 2019 1:49 AM
To: Li, Redhairer <redhairer.li(a)intel.com>
Cc: linux-nvdimm(a)lists.01.org
Subject: Re: [PATCH] daxctl: Change region input type from INTEGER to STRING.
On Wed, Dec 25, 2019 at 6:22 PM Li, Redhairer <redhairer.li(a)intel.com> wrote:
Build error is solved by following change.
diff --git a/test/Makefile.am b/test/Makefile.am index
829146d..d764190 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -98,10 +98,10 @@ ack_shutdown_count_set_SOURCES =\
ack_shutdown_count_set_LDADD = $(LIBNDCTL_LIB) $(KMOD_LIBS)
blk_ns_SOURCES = blk_namespaces.c $(testcore) -blk_ns_LDADD =
$(LIBNDCTL_LIB) $(KMOD_LIBS)
+blk_ns_LDADD = $(LIBNDCTL_LIB) $(KMOD_LIBS) $(UUID_LIBS)
pmem_ns_SOURCES = pmem_namespaces.c $(testcore) -pmem_ns_LDADD =
$(LIBNDCTL_LIB) $(KMOD_LIBS)
+pmem_ns_LDADD = $(LIBNDCTL_LIB) $(KMOD_LIBS) $(UUID_LIBS)
dpa_alloc_SOURCES = dpa-alloc.c $(testcore) dpa_alloc_LDADD =
$(LIBNDCTL_LIB) $(UUID_LIBS) $(KMOD_LIBS) @@ -143,6 +143,7 @@
device_dax_LDADD = \
$(LIBNDCTL_LIB) \
$(KMOD_LIBS) \
$(JSON_LIBS) \
+ $(UUID_LIBS) \
../libutil.a
Ah, sorry about that.
Hmm, are you running this on top of Ubuntu?
We have had a long standing problem whereby Fedora autoincludes some library dependencies
and Ubuntu does not.
011fc692270b ndctl, test: add UUID_LIBS for list_smart_dimm
Please send this as a formal fix patch.