From daniel.thompson at linaro.org  Thu Oct  5 14:33:57 2017
From: daniel.thompson at linaro.org (Daniel Thompson)
Date: Thu,  5 Oct 2017 22:33:57 +0100
Subject: [PATCH] test-patchset: Make a copy of const_structs.checkpatch
Message-ID: <20171005213357.4369-1-daniel.thompson@linaro.org>

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>
---
 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



From jacob.e.keller at intel.com  Thu Oct  5 15:25:57 2017
From: jacob.e.keller at intel.com (Keller, Jacob E)
Date: Thu, 5 Oct 2017 22:25:57 +0000
Subject: [PATCH] test-patchset: Make a copy of const_structs.checkpatch
In-Reply-To: <20171005213357.4369-1-daniel.thompson@linaro.org>
References: <20171005213357.4369-1-daniel.thompson@linaro.org>
Message-ID: <02874ECE860811409154E81DA85FBB5882ABC4C0@ORSMSX115.amr.corp.intel.com>



> -----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



