[LTP] [PATCH] .b4-config: Add checkpatch.pl default commands

Andrea Cervesato andrea.cervesato@suse.com
Wed Jan 15 10:10:20 CET 2025


Hi again,

On 1/14/25 23:43, Ricardo B. Marliere via ltp wrote:
> From: Ricardo B. Marliere <rbm@suse.com>
>
> Blend b4 default checkpatch.pl flags with the ones defined in
> include/mk/env_post.mk in .b4-config so that a contributor may use `b4 prep
> --check` and `b4 am --check` in his workflow.
>
> Signed-off-by: Ricardo B. Marliere <rbm@suse.com>
> ---
>   .b4-config | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/.b4-config b/.b4-config
> index 2efdf2273240dcf57bd5cab174ff6c7a1952dd5b..5a16bd64ec98b91e17386c0eadc7ef0de54d86f3 100644
> --- a/.b4-config
> +++ b/.b4-config
> @@ -4,3 +4,6 @@
>       send-series-to = Linux Test Project <ltp@lists.linux.it>
>       pw-url = https://patchwork.ozlabs.org/
>       pw-project = ltp
> +    prep-perpatch-check-cmd = ./scripts/checkpatch.pl -q --terse --no-summary --mailback --showfile --no-tree --ignore CONST_STRUCT,VOLATILE,SPLIT_STRING
According to checkpatch.pl documentation [1] , we should probably ignore 
the FILE_PATH_CHANGES here, since MAINTAINERS file is not splitting any 
responsibility towards LTP folders and maintenance. After disabling this 
check, warnings around file add/move/delete disappears.
> +    am-perpatch-check-cmd = ./scripts/checkpatch.pl -q --terse --no-summary --mailback --no-tree --ignore CONST_STRUCT,VOLATILE,SPLIT_STRING
> +
>
> ---
> base-commit: 6fe8aa186559784f0394cd449cba6c53342790ec
> change-id: 20250114-fix_b4-config-1ab84320000a
>
> Best regards,
Andrea


More information about the ltp mailing list