From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id EDF5CA0A02; Fri, 15 Jan 2021 12:59:14 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BD00B141015; Fri, 15 Jan 2021 12:59:14 +0100 (CET) Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by mails.dpdk.org (Postfix) with ESMTP id C3A4214100D for ; Fri, 15 Jan 2021 12:59:12 +0100 (CET) IronPort-SDR: 2lshtPiGVwvxEYawVuyaUPzJBORgixn47xaKEbltUR7dxxymPm+yugZa1P6TF0+Rjju4PKdBKv iWG9Hdg3k7hg== X-IronPort-AV: E=McAfee;i="6000,8403,9864"; a="157717311" X-IronPort-AV: E=Sophos;i="5.79,349,1602572400"; d="scan'208";a="157717311" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Jan 2021 03:59:11 -0800 IronPort-SDR: 4ZsDeJsY5wLiflJKOolkP34QyXDmvXlkPfiHSBE6STlhTu69TsagZgZWTzztbo7u6vrZvf9aOj iofin5DA8Hqg== X-IronPort-AV: E=Sophos;i="5.79,349,1602572400"; d="scan'208";a="425289907" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.8.25]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 15 Jan 2021 03:59:08 -0800 Date: Fri, 15 Jan 2021 11:59:05 +0000 From: Bruce Richardson To: Ferruh Yigit Cc: dev@dpdk.org, david.marchand@redhat.com, Thomas Monjalon , Andrew Rybchenko , Yipeng Wang , Sameh Gobriel , Konstantin Ananyev , Bernard Iremonger , Vladimir Medvedkin , Ori Kam , Honnappa Nagarahalli , Olivier Matz , Cristian Dumitrescu Message-ID: <20210115115905.GB1487@bricha3-MOBL.ger.corp.intel.com> References: <20210114110606.21142-1-bruce.richardson@intel.com> <20210115111052.16437-1-bruce.richardson@intel.com> <20210115111052.16437-17-bruce.richardson@intel.com> <9b696a65-2012-7ae8-d840-1515f8bf140b@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9b696a65-2012-7ae8-d840-1515f8bf140b@intel.com> Subject: Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Jan 15, 2021 at 11:51:49AM +0000, Ferruh Yigit wrote: > On 1/15/2021 11:10 AM, Bruce Richardson wrote: > > To verify that all DPDK headers are ok for inclusion directly in a C > > file, and are not missing any other pre-requisite headers, we can > > auto-generate for each header an empty C file that includes that header. > > Compiling these files will throw errors if any header has unmet > > dependencies. > > > > The list of headers to check is based of the "headers" value returned from > > each library's meson.build file. However, since not all headers are for > > direct inclusion, add a build variable "headers_no_chkincs" to list those > > headers and skip checking them. > > > > Signed-off-by: Bruce Richardson > > --- > > > > v2: > > * add maintainers entry > > * distribute exception list among meson.build files. > > > > MAINTAINERS | 4 ++++ > > app/chkincs/gen_c_file_for_header.py | 12 ++++++++++ > > app/chkincs/main.c | 4 ++++ > > app/chkincs/meson.build | 28 ++++++++++++++++++++++++ > > +1 to have this kind of tool to check, but it is not an application like > others in the 'app' folder, what do you think placing it under 'devtools' or > 'buildtools'? Couple of reasons why it's placed in app. 1. We previously had a "chkincs" app in DPDK which was kept in the app folder 2. It allows us to reuse the build infrastructure for building apps, rather than reduplicating it. 3. We don't have any compilable code currently in the devtools folder, and even in buildtools the pmdinfogen app is going to go away. That being said, none of those reasons are major issues that can't be worked around if the consensus is to move it. /Bruce