From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 4ECE95584 for ; Fri, 15 Jul 2016 23:03:21 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga103.fm.intel.com with ESMTP; 15 Jul 2016 14:03:06 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,369,1464678000"; d="scan'208";a="734896201" Received: from unknown ([10.252.22.155]) by FMSMGA003.fm.intel.com with SMTP; 15 Jul 2016 14:03:03 -0700 Received: by (sSMTP sendmail emulation); Fri, 15 Jul 2016 22:03:03 +0025 Date: Fri, 15 Jul 2016 22:03:02 +0100 From: Bruce Richardson To: Adrien Mazarguil Cc: dev@dpdk.org, Ferruh Yigit Message-ID: <20160715210302.GA52536@bricha3-MOBL3> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH v4 00/10] Fix build errors related to exported headers X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Jul 2016 21:03:21 -0000 On Wed, Jul 13, 2016 at 03:02:37PM +0200, Adrien Mazarguil wrote: > DPDK uses GNU C language extensions in most of its code base. This is fine > for internal source files whose compilation flags are controlled by DPDK, > however user applications that use exported "public" headers may experience > compilation failures when enabling strict error/standard checks (-std and > -pedantic for instance). > > Exported headers are installed system-wide and must be as clean as possible > so applications do not have to resort to workarounds. > > This patchset affects exported headers only, compilation problems are > addressed as follows: > > - Adding the __extension__ keyword to nonstandard constructs (same method > as existing libraries when there is no other choice). > - Adding the __extension__ keyword to C11 constructs to remain compatible > with pure C99. > - Adding missing includes so exported files can be included out of order > and on their own. > - Fixing GNU printf-like variadic macros as there is no magic keyword for > these. > Having upgraded to Fedora 24, I'm seeing quite a few errors compiling with gcc 6.1.1 in debug mode. Applying this patchset seems to really cut down on those errors, so may need to be applied for 16.07 release. The remaining error I'm seeing is, in mlx drivers, complaints about the pedantic flag (the flag which I think was causing all the other errors to be triggered too): error: `-pedantic' is not an option that controls warnings For this set though, I don't see any new errors introduced into gcc or clang builds for the libs or drivers, and a number of errors cleared, so: Tested-by: Bruce Richardson Regards, /Bruce