From mboxrd@z Thu Jan 1 00:00:00 1970 From: Neil Horman Subject: Re: mk: fix app linking for combined libs Date: Mon, 8 Dec 2014 09:53:58 -0500 Message-ID: <20141208145358.GG3237@localhost.localdomain> References: <1414078604-1090-1-git-send-email-sergio.gonzalez.monroy@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev-VfR2kkLFssw@public.gmane.org To: Sergio Gonzalez Monroy Return-path: Content-Disposition: inline In-Reply-To: <1414078604-1090-1-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On Thu, Oct 23, 2014 at 04:36:44PM +0100, Sergio Gonzalez Monroy wrote: > Building combined shared libraries results in applications being linked > against separeted/individual and combined libs altogether. > > Link only against combined lib when the config option is enabled. > > Signed-off-by: Sergio Gonzalez Monroy > Acked-by: Pablo de Lara > > --- > mk/rte.app.mk | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/mk/rte.app.mk b/mk/rte.app.mk > index 285b65c..45444af 100644 > --- a/mk/rte.app.mk > +++ b/mk/rte.app.mk > @@ -217,6 +217,12 @@ endif > > endif # plugins > > +ifeq ($(RTE_BUILD_COMBINE_LIBS),y) > +LDLIBS = --whole-archive > +LDLIBS += --start-group > +LDLIBS += -l$(RTE_LIBNAME) > +endif > + > LDLIBS += $(EXECENV_LDLIBS) > > LDLIBS += --end-group > @@ -240,10 +246,6 @@ build: _postbuild > > exe2cmd = $(strip $(call dotfile,$(patsubst %,%.cmd,$(1)))) > > -ifeq ($(RTE_BUILD_COMBINE_LIBS),y) > -LDLIBS += -l$(RTE_LIBNAME) > -endif > - > ifeq ($(LINK_USING_CC),1) > LDLIBS := $(call linkerprefix,$(LDLIBS)) > LDFLAGS := $(call linkerprefix,$(LDFLAGS)) Acked-by: Neil Horman