From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH v2] mk: fix eal shared library dependencies -lrt Date: Tue, 22 Mar 2016 14:18:10 +0200 Message-ID: <56F13802.60206@redhat.com> References: <1458634142-10084-1-git-send-email-danielx.t.mrzyglod@intel.com> <1458640699-8108-1-git-send-email-danielx.t.mrzyglod@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit To: Daniel Mrzyglod , dev@dpdk.org Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 4731329C6 for ; Tue, 22 Mar 2016 13:18:12 +0100 (CET) In-Reply-To: <1458640699-8108-1-git-send-email-danielx.t.mrzyglod@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 03/22/2016 11:58 AM, Daniel Mrzyglod wrote: > For GLIBC < 2.17 it is necessery to add -lrt for linker > from glibc > 2.17 The `clock_*' suite of functions (declared in ) is now > available directly in the main C library. This affect Ubuntu 12.04 in i686 > and other older Linux Distros). > > Fixes: 4758404a3084 ("mk: fix eal shared library dependencies") > > Signed-off-by: Daniel Mrzyglod > --- > app/test/Makefile | 1 + > examples/ptpclient/Makefile | 1 + > lib/librte_eal/linuxapp/eal/Makefile | 1 + > 3 files changed, 3 insertions(+) > > diff --git a/app/test/Makefile b/app/test/Makefile > index 00e4df2..707930f 100644 > --- a/app/test/Makefile > +++ b/app/test/Makefile > @@ -114,6 +114,7 @@ SRCS-$(CONFIG_RTE_LIBRTE_CMDLINE) += test_cmdline_string.c > SRCS-$(CONFIG_RTE_LIBRTE_CMDLINE) += test_cmdline_lib.c > > ifeq ($(CONFIG_RTE_LIBRTE_SCHED),y) > +LDFLAGS += -lrt > SRCS-y += test_red.c > SRCS-y += test_sched.c > endif > diff --git a/examples/ptpclient/Makefile b/examples/ptpclient/Makefile > index b77cf71..082c690 100644 > --- a/examples/ptpclient/Makefile > +++ b/examples/ptpclient/Makefile > @@ -46,6 +46,7 @@ SRCS-y := ptpclient.c > > CFLAGS += -O3 > CFLAGS += $(WERROR_FLAGS) > +LDFLAGS += -lrt These two should be LDLIBS instead. LDFLAGS is for passing in other arguments to the linker, such as path, LDLIBS exists solely for passing in the actual libraries. - Panu -