[PATCH v5 01/50] devtools: script to remove unused headers includes

Thomas Monjalon thomas at monjalon.net
Thu Jan 20 16:00:58 CET 2022


17/01/2022 21:18, Sean Morrissey:
> This script can be used for removing headers flagged for removal by the
> include-what-you-use (IWYU) tool. The script has the ability to remove
> headers from specified sub-directories or dpdk as a whole and tests the
> build after each removal by calling meson compile.
> 
> example usages:
> 
> Remove headers flagged by iwyu_tool output file
> $ ./devtools/process_iwyu.py iwyu.out -b build
> 
> Remove headers flagged by iwyu_tool output file from sub-directory
> $ ./devtools/process_iwyu.py iwyu.out -b build -d lib/kvargs
> 
> Remove headers directly piped from the iwyu_tool
> $ iwyu_tool -p build | ./devtools/process_iwyu.py - -b build

Can it be an automatic check per patch?
Or are we supposed to run it from time to time?

> Signed-off-by: Sean Morrissey <sean.morrissey at intel.com>
> Signed-off-by: Conor Fogarty <conor.fogarty at intel.com>
> 
> Reviewed-by: Bruce Richardson <bruce.richardson at intel.com>

Please no blank line between names.

I assume you checked with flake8?





More information about the dev mailing list