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:15:43 +0100 Message-ID: <3082304.pESdSe8qVs@xps> References: <20190103175725.5836-1-bluca@debian.org> <20190206170848.19059-1-bluca@debian.org> <20190206170848.19059-2-bluca@debian.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, bruce.richardson@intel.com To: Luca Boccassi Return-path: Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id B102E1B426 for ; Tue, 12 Feb 2019 12:15:50 +0100 (CET) In-Reply-To: <20190206170848.19059-2-bluca@debian.org> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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