Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-20 Thread David Marchand
On Wed, Jan 20, 2021 at 3:34 PM Bruce Richardson wrote: > On Fri, Jan 15, 2021 at 02:59:08PM +, Bruce Richardson wrote: > > On Fri, Jan 15, 2021 at 02:55:41PM +, Bruce Richardson wrote: > > > On Fri, Jan 15, 2021 at 03:09:25PM +0100, Thomas Monjalon wrote: > > > I'm honestly fine either wa

Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-20 Thread Bruce Richardson
On Fri, Jan 15, 2021 at 02:59:08PM +, Bruce Richardson wrote: > On Fri, Jan 15, 2021 at 02:55:41PM +, Bruce Richardson wrote: > > On Fri, Jan 15, 2021 at 03:09:25PM +0100, Thomas Monjalon wrote: > > > 15/01/2021 12:59, Bruce Richardson: > > > > On Fri, Jan 15, 2021 at 11:51:49AM +, Ferr

Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-15 Thread Bruce Richardson
On Fri, Jan 15, 2021 at 02:55:41PM +, Bruce Richardson wrote: > On Fri, Jan 15, 2021 at 03:09:25PM +0100, Thomas Monjalon wrote: > > 15/01/2021 12:59, Bruce Richardson: > > > On Fri, Jan 15, 2021 at 11:51:49AM +, Ferruh Yigit wrote: > > > > On 1/15/2021 11:10 AM, Bruce Richardson wrote: > >

Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-15 Thread Bruce Richardson
On Fri, Jan 15, 2021 at 03:09:25PM +0100, Thomas Monjalon wrote: > 15/01/2021 12:59, Bruce Richardson: > > On Fri, Jan 15, 2021 at 11:51:49AM +, 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

Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-15 Thread Thomas Monjalon
15/01/2021 12:59, Bruce Richardson: > On Fri, Jan 15, 2021 at 11:51:49AM +, 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 >

Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-15 Thread Bruce Richardson
On Fri, Jan 15, 2021 at 11:51:49AM +, 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

Re: [dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-15 Thread Ferruh Yigit
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

[dpdk-dev] [PATCH v2 16/19] app/chkincs: add chkincs app to verify headers

2021-01-15 Thread Bruce Richardson
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