From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <nhorman@tuxdriver.com>
Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58])
 by dpdk.org (Postfix) with ESMTP id 570C0959
 for <dev@dpdk.org>; Tue, 16 Dec 2014 14:58:37 +0100 (CET)
Received: from hmsreliant.think-freely.org
 ([2001:470:8:a08:7aac:c0ff:fec2:933b] helo=localhost)
 by smtp.tuxdriver.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.63)
 (envelope-from <nhorman@tuxdriver.com>)
 id 1Y0sdv-0005Rx-T2; Tue, 16 Dec 2014 08:58:30 -0500
Date: Tue, 16 Dec 2014 08:58:22 -0500
From: Neil Horman <nhorman@tuxdriver.com>
To: Thomas Monjalon <thomas.monjalon@6wind.com>
Message-ID: <20141216135822.GC13806@hmsreliant.think-freely.org>
References: <2438199.ui6ep4sFDa@xps13>
 <1418684684-23642-1-git-send-email-thomas.monjalon@6wind.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1418684684-23642-1-git-send-email-thomas.monjalon@6wind.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
X-Spam-Score: -2.9 (--)
X-Spam-Status: No
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v2] mk: fix build with shared pcap pmd
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Tue, 16 Dec 2014 13:58:37 -0000

On Tue, Dec 16, 2014 at 12:04:44AM +0100, Thomas Monjalon wrote:
> Some applications doesn't have the pcap link flag
> when shared libraries are enabled.
> Indeed in such case, pcap PMD must not be linked but pcap library should.
> 
> Actually -lpcap is always needed if pcap PMD is used,
> and -lrte_pmd_pcap must be set only with static PMD library.
> So the flags -lrte_pmd_pcap and -lpcap are enabled separately.
> 
> Workarounds in test-pmd/ and test-pipeline/ can be removed.
> 
> Reported-by: Stepan Sojka <stepan.sojka@adaptivemobile.com>
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> ---
> 
> v2:
> fix in rte.app.mk
> 
> v1 from Stepan:
> fix every applications
> 
>  app/test-pipeline/Makefile | 4 ----
>  app/test-pmd/Makefile      | 4 ----
>  mk/rte.app.mk              | 6 +++++-
>  3 files changed, 5 insertions(+), 9 deletions(-)
> 
> diff --git a/app/test-pipeline/Makefile b/app/test-pipeline/Makefile
> index b81652f..aa6df0c 100644
> --- a/app/test-pipeline/Makefile
> +++ b/app/test-pipeline/Makefile
> @@ -41,10 +41,6 @@ APP = testpipeline
>  CFLAGS += -O3
>  CFLAGS += $(WERROR_FLAGS)
>  
> -ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),y)
> -LDFLAGS += -lpcap
> -endif
> -
>  #
>  # all source are stored in SRCS-y
>  #
> diff --git a/app/test-pmd/Makefile b/app/test-pmd/Makefile
> index 97dc2e6..dcf26f4 100644
> --- a/app/test-pmd/Makefile
> +++ b/app/test-pmd/Makefile
> @@ -41,10 +41,6 @@ APP = testpmd
>  CFLAGS += -O3
>  CFLAGS += $(WERROR_FLAGS)
>  
> -ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),y)
> -LDFLAGS += -lpcap
> -endif
> -
>  #
>  # all source are stored in SRCS-y
>  #
> diff --git a/mk/rte.app.mk b/mk/rte.app.mk
> index 84ec4df..c5eaf82 100644
> --- a/mk/rte.app.mk
> +++ b/mk/rte.app.mk
> @@ -119,6 +119,10 @@ LDLIBS += -lm
>  LDLIBS += -lrt
>  endif
>  
> +ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),y)
> +LDLIBS += -lpcap
> +endif
> +
>  LDLIBS += --start-group
>  
>  ifeq ($(CONFIG_RTE_LIBRTE_KVARGS),y)
> @@ -207,7 +211,7 @@ LDLIBS += -lrte_pmd_ring
>  endif
>  
>  ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),y)
> -LDLIBS += -lrte_pmd_pcap -lpcap
> +LDLIBS += -lrte_pmd_pcap
>  endif
>  
>  ifeq ($(CONFIG_RTE_LIBRTE_PMD_AF_PACKET),y)
> -- 
> 2.1.3
> 
> 

Actually, what if we just add $(LDFLAGS) to the O_TO_S rule in mk/rte.lib.mk?
Then in lib/librte_pmd_pcap/Makefile, we can just add LDFLAGS+=-lpcap, and the
loading of the pcap pmd will itself require the loading of libpcap.  That would
be a nice clean implementation that allows applications to just link the pmd and
not have to worry about dependencies.  It would also allow us to clean up other
dependencies like the xenvirt pmd and vhost.

Neil