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

Ferruh Yigit ferruh.yigit at intel.com
Fri Jan 15 12:51:49 CET 2021


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 <bruce.richardson at intel.com>
> ---
> 
> 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'?


More information about the dev mailing list