[dpdk-dev] [PATCH 1/2] scripts: add checkpatch wrapper

David Marchand david.marchand at 6wind.com
Thu Oct 29 13:33:59 CET 2015


On Fri, Oct 23, 2015 at 5:53 PM, Thomas Monjalon <thomas.monjalon at 6wind.com>
wrote:

> +for p in "$@" ; do
> +       printf -- "\n### $p\n\n"
> +       report=$($DPDK_CHECKPATCH_PATH $options "$p" 2>/dev/null)
> +       [ $? -ne 0 ] || continue
> +       printf '%s\n' "$report" | head -n -6
> +       status=$(($status + 1))
> +done
> +exit $status
>

I prefer when checking scripts only complain when something is wrong :-)
So I would only display the file name if checkpatch complains.

-- 
David Marchand


More information about the dev mailing list