[PATCH] test-patchset: Make a copy of const_structs.checkpatch

Keller, Jacob E jacob.e.keller at intel.com
Thu Oct 5 15:25:57 PDT 2017



> -----Original Message-----
> From: Daniel Thompson [mailto:daniel.thompson at linaro.org]
> Sent: Thursday, October 05, 2017 2:34 PM
> To: Keller, Jacob E <jacob.e.keller at intel.com>
> Cc: Daniel Thompson <daniel.thompson at linaro.org>; aiaiai at lists.infradead.org;
> patches at linaro.org; Lee Jones <lee.jones at linaro.org>
> Subject: [PATCH] test-patchset: Make a copy of const_structs.checkpatch
> 
> Currently aiaiai issues the following warning for each patch under test
> because recent versions of checkpatch look for a list of structures in
> scripts/const_structs.checkpatch .
> 
> This patch fixes the problem by copying out the extra file (if one is
> found).
> 
> Signed-off-by: Daniel Thompson <daniel.thompson at linaro.org>

Applied.

Thanks,
Jake

> ---
>  aiaiai-test-patchset | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
> 
> diff --git a/aiaiai-test-patchset b/aiaiai-test-patchset
> index 6193dfcae0f1..0ecbac68e9fd 100755
> --- a/aiaiai-test-patchset
> +++ b/aiaiai-test-patchset
> @@ -481,12 +481,18 @@ if [ -n "$checkpatch" ]; then
>  		git show "$commit_id1:scripts/checkpatch.pl" > "$checkpatch_pl"
>  		chmod $verbose u+x "$checkpatch_pl" >&2
> 
> -		# Also grab the typo corrections if the kernel under test has
> +		# Grab the typo corrections if the kernel under test has
>  		# one (no need to report errors... if something is weird
>  		# checkpatch will let us know).
>  		if git cat-file -e "$commit_id1:scripts/spelling.txt" 2>/dev/null;
> then
>  			git show "$commit_id1:scripts/spelling.txt" >
> "$tmpdir/checkpatch/spelling.txt"
>  		fi
> +
> +		# Also grab the list of constant structures (if present in this
> kernel)
> +		if git cat-file -e "$commit_id1:scripts/const_structs.checkpatch"
> 2>/dev/null; then
> +			git show "$commit_id1:scripts/const_structs.checkpatch"
> > "$tmpdir/checkpatch/const_structs.checkpatch"
> +		fi
> +
>  	else
>  		verbose "Can't find checkpatch.pl.. disabling checkpatch tests."
>  		checkpatch=
> --
> 2.9.5




More information about the aiaiai mailing list