From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v6 2/5] build: use dependency() instead of find_library() Date: Tue, 12 Feb 2019 12:36:17 +0100 Message-ID: <3010506.HFqD7W5Z4o@xps> References: <20190103175725.5836-1-bluca@debian.org> <3082304.pESdSe8qVs@xps> <20190212113154.GA517812@bricha3-MOBL.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Luca Boccassi , dev@dpdk.org To: Bruce Richardson Return-path: Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id B51C51B469 for ; Tue, 12 Feb 2019 12:36:19 +0100 (CET) In-Reply-To: <20190212113154.GA517812@bricha3-MOBL.ger.corp.intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 12/02/2019 12:31, Bruce Richardson: > On Tue, Feb 12, 2019 at 12:15:43PM +0100, Thomas Monjalon wrote: > > 06/02/2019 18:08, Luca Boccassi: > > > Whenever possible (if the library ships a pkg-config file) use meson's > > > dependency() function to look for it, as it will automatically add it > > > to the Requires.private list if needed, to allow for static builds to > > > succeed for reverse dependencies of DPDK. Otherwise the recursive > > > dependencies are not parsed, and users doing static builds have to > > > resolve them manually by themselves. > > > When using this API avoid additional checks that are superfluous and > > > take extra time, and avoid adding the linker flag manually which causes > > > it to be duplicated. > > > > > > Signed-off-by: Luca Boccassi > > > Acked-by: Bruce Richardson > > > > This patch breaks compilation on my machine with a lot of strange errors: > > > > from drivers/net/pcap/rte_eth_pcap.c > > > > /usr/include/stdint.h:109: error: "__INT64_C" redefined > > /usr/include/bits/stdlib-bsearch.h:32:13: error: cast discards 'const' qualifier from pointer target type > > /usr/include/pthread.h:682:6: error: 'regparm' attribute directive ignored > > etc > > > Is this on Arch linux again? I just reverified these first two patches and > they work fine for me on Fedora (I assume Luca probably tested them already on > Debian) > > Anything unusual about your setup? Nothing unusual. Just using the best environment ever ;) I will try to debug it.