From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 550898042 for ; Mon, 8 Dec 2014 16:02:10 +0100 (CET) Received: from rrcs-70-62-112-196.midsouth.biz.rr.com ([70.62.112.196] helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63) (envelope-from ) id 1Xxzoz-0000zP-SU; Mon, 08 Dec 2014 10:02:04 -0500 Date: Mon, 8 Dec 2014 10:01:54 -0500 From: Neil Horman To: Hiroshi Shimamoto Message-ID: <20141208150154.GA3907@localhost.localdomain> References: <7F861DC0615E0C47A872E6F3C5FCDDBD05DCB8C6@BPXM14GP.gisp.nec.co.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7F861DC0615E0C47A872E6F3C5FCDDBD05DCB8C6@BPXM14GP.gisp.nec.co.jp> User-Agent: Mutt/1.5.23 (2014-03-12) X-Spam-Score: -2.9 (--) X-Spam-Status: No Cc: "dev@dpdk.org" , Hayato Momma Subject: Re: [dpdk-dev] mk: fix LDFLAGS for shared lib X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Dec 2014 15:02:10 -0000 On Wed, Dec 03, 2014 at 11:33:54PM +0000, Hiroshi Shimamoto wrote: > From: Hiroshi Shimamoto > > Only CPU_LDFLAGS is used in mk/rte.sharelib.mk. > It should be LDFLAGS to build the library with correct linkage options. > > Signed-off-by: Hiroshi Shimamoto > Reviewed-by: Hayato Momma > > --- > mk/rte.sharelib.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mk/rte.sharelib.mk b/mk/rte.sharelib.mk > index c0a811a..df6c268 100644 > --- a/mk/rte.sharelib.mk > +++ b/mk/rte.sharelib.mk > @@ -45,7 +45,7 @@ sharelib: $(LIB_ONE) FORCE > > OBJS = $(wildcard $(RTE_OUTPUT)/build/lib/*.o) > > -O_TO_S = $(LD) $(CPU_LDFLAGS) -shared $(OBJS) -o $(RTE_OUTPUT)/lib/$(LIB_ONE) > +O_TO_S = $(LD) $(LDFLAGS) -shared $(OBJS) -o $(RTE_OUTPUT)/lib/$(LIB_ONE) > O_TO_S_STR = $(subst ','\'',$(O_TO_S)) #'# fix syntax highlight > O_TO_S_DISP = $(if $(V),"$(O_TO_S_STR)"," LD $(@)") > O_TO_S_CMD = "cmd_$@ = $(O_TO_S_STR)" Does this continue to work on cross compiled arches (32 bit on 64 bit systems and vice versa)? I have this recollection that something in CPU_LDFLAGS was needed Neil