From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] app/testpmd: fix build without ixgbe Date: Thu, 15 Jun 2017 14:48:32 +0200 Message-ID: <5294711.b1ZTK55eI0@xps> References: <20170615095446.2187-1-thomas@monjalon.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: wenzhuo.lu@intel.com, dev@dpdk.org To: Radu Nicolau Return-path: Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id DCAF723B for ; Thu, 15 Jun 2017 14:48:33 +0200 (CEST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 15/06/2017 13:37, Radu Nicolau: > Looks ok to me, but why would one enable IXGBE_BYPASS without enabling > IXGBE? [...] > > -#ifdef RTE_LIBRTE_IXGBE_BYPASS > > +#if defined RTE_LIBRTE_IXGBE_PMD && defined RTE_LIBRTE_IXGBE_BYPASS RTE_LIBRTE_IXGBE_BYPASS should not be enabled if RTE_LIBRTE_IXGBE_PMD is disabled. But it can happen in some scripted configuration like test-build.sh does. Anyway we should drop the RTE_LIBRTE_IXGBE_BYPASS configuration option and enables it always.