From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [dpdk-stable] [PATCH v5 1/2] mk: fix cross build errors Date: Wed, 30 May 2018 12:18:14 +0200 Message-ID: <4579229.sggoYGe59P@xps> References: <3248809.BvU3dfXijd@xps> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Bruce Richardson , "dev@dpdk.org" To: Gavin Hu Return-path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 4F27237B0 for ; Wed, 30 May 2018 12:18:17 +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" 30/05/2018 11:41, Gavin Hu: > From: Thomas Monjalon > > 30/05/2018 04:45, Gavin Hu: > > > From: Bruce Richardson > > > > On Tue, May 29, 2018 at 04:45:55PM +0200, Thomas Monjalon wrote: > > > > > 28/05/2018 15:24, Bruce Richardson: > > > > > > Would a simpler solution for this not be to put "-Wno-implicit- > > > > fallthrough" > > > > > > for pmdinfogen? GCC will not give a warning for an unrecognised "- > > Wno" > > > > > > flag when compiling, unless there are other errors. This means > > > > > > we can just use the flag without bothering with version checks. > > > > > > > > > > No, it does not work. > > > > > I have this error with clang 5.0.1: > > > > > error: unknown warning option '-Wno-format-truncation' > > > > > > > > > Yes, you still need to check for GCC to use the flag, just not for a > > > > specific version of GCC. > > > > > > > > /Bruce > > > [Gavin Hu] Hi Thomas, '-Wno-format-truncation' is applied to gcc only, not > > to clang. > > > Gcc, icc and clang have their own WERROR_FLAGS, they should not mix up > > with each other. > > > > Now I understand what we missed from the beginning: I am testing with > > meson. > > This flag is added unconditionnaly in config/meson.build. > > [Gavin Hu] Hi Thomas and Bruce, then do you have more comments on the v5 patch? > Any comments are welcome, if no, can you act the patches for merging? Of course we have comments! I keep telling you that compilation is not working. Try meson compilation: devtools/test-meson-builds.sh