[dpdk-dev,v2] vhost: Add -lfuse into the LDFLAGS list

Message ID 1422369594-25960-1-git-send-email-nhorman@tuxdriver.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Neil Horman Jan. 27, 2015, 2:39 p.m. UTC
  the vhost library relies on libfuse, and thats included when we do a normal
shared object build, but when we specify combined libs, its gets left out.  Add
it back in

Signed-off-by: Neil Horman <nhorman@tuxdriver.com>

---
Change notes:
v2) Removed normal shared object inclusion of libfuse since its always included
now
---
 mk/rte.app.mk | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
  

Comments

Thomas Monjalon Jan. 27, 2015, 4:02 p.m. UTC | #1
2015-01-27 09:39, Neil Horman:
> the vhost library relies on libfuse, and thats included when we do a normal
> shared object build, but when we specify combined libs, its gets left out.  Add
> it back in
> 
> Signed-off-by: Neil Horman <nhorman@tuxdriver.com>
> 
> ---
> Change notes:
> v2) Removed normal shared object inclusion of libfuse since its always included
> now

Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Applied

Thanks
  

Patch

diff --git a/mk/rte.app.mk b/mk/rte.app.mk
index 9c8b06a..4294d9a 100644
--- a/mk/rte.app.mk
+++ b/mk/rte.app.mk
@@ -131,6 +131,10 @@  ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),y)
 LDLIBS += -lpcap
 endif
 
+ifeq ($(CONFIG_RTE_LIBRTE_VHOST),y)
+LDLIBS += -lfuse
+endif
+
 LDLIBS += --start-group
 
 ifeq ($(CONFIG_RTE_BUILD_COMBINE_LIBS),n)
@@ -197,7 +201,6 @@  endif
 
 ifeq ($(CONFIG_RTE_LIBRTE_VHOST), y)
 LDLIBS += -lrte_vhost
-LDLIBS += -lfuse
 endif
 
 ifeq ($(CONFIG_RTE_LIBRTE_ENIC_PMD),y)