ci: Use --ACM --no-renames

When collecting changed files, we don't care about rename detection,
only filtering-out deleted deleting files.

Signed-off-by: Jorge Marques <[email protected]>

Author: gastmaier

ci/build.sh

@@ -18,7 +18,13 @@ check_checkpatch() {
 		git --no-pager show --format="%h %s" "$commit" --name-only
 		# Skip empty commits, assume cover letter
 		# and those only touching non-upstream directories .github ci and docs
-		local files=$(git diff --diff-filter=ACMR --name-only $commit~..$commit | grep -v ^ci | grep -v ^.github | grep -v ^docs || true)
+		# A treeless (--filter=tree:0) fetch could be done to fetch
+		# full commit message history before running checkpatch, but
+		# that may mess-up the cache and is only useful for checking
+		# SHA references in the commit message, with may not even apply
+		# if the commit is from upstream. Instead, just delegate to the
+		# user to double check the referenced SHA.
+		local files=$(git diff --diff-filter=ACM --no-renames --name-only $commit~..$commit | grep -v ^ci | grep -v ^.github | grep -v ^docs || true)
 		if [[ -z "$files" ]]; then
 			echo "empty, skipped"
 			continue
@@ -114,7 +120,7 @@ check_checkpatch() {
 }
 
 check_dt_binding_check() {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local fail=0
 
 	echo "dt_binding_check on range $base_sha..$head_sha"
@@ -164,7 +170,7 @@ check_dt_binding_check() {
 }
 
 check_coccicheck() {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local mail=
 	local warn=0
 
@@ -241,7 +247,7 @@ check_coccicheck() {
 }
 
 check_cppcheck () {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local regex='^[[:alnum:]/._-]+:[[:digit:]]+:[[:digit:]]+: .*$'
 	local mail=
 	local fail=0
@@ -335,7 +341,7 @@ compile_devicetree() {
 
 	echo "compile devicetree on range $base_sha..$head_sha"
 
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local dtsi_files=$(echo "$files" | grep ^arch/$ARCH/boot/dts/ | grep dtsi$ || true)
 	if [[ ! -z "$dtsi_files" ]]; then
 		echo "collecting dts files that include dtsi"
@@ -644,7 +650,7 @@ compile_kernel_smatch() {
 }
 
 compile_gcc_fanalyzer () {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local regex='^[[:alnum:]/._-]+:[[:digit:]]+:[[:digit:]]+: .*$'
 	local mail=
 	local fail=0
@@ -731,7 +737,7 @@ compile_gcc_fanalyzer () {
 }
 
 compile_clang_analyzer () {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local regex='^[[:alnum:]/._-]+:[[:digit:]]+:[[:digit:]]+: .*$'
 	local mail=
 	local fail=0
@@ -826,7 +832,7 @@ compile_clang_analyzer () {
 }
 
 assert_compiled () {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	local fail=0
 
 	echo "assert sources were compiled on range $base_sha..$head_sha"
@@ -865,7 +871,7 @@ apply_prerun() {
 	# e.g. manipulate the source code depending on run conditons or target.
 	local coccis=$(ls ci/prerun/*.cocci 2>/dev/null)
 	local bashes=$(ls ci/prerun/*.sh 2>/dev/null)
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 
 	echo "apply_prerun on range $base_sha..$head_sha"
 
@@ -894,13 +900,13 @@ apply_prerun() {
 }
 
 touch_files () {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 
 	touch $files
 }
 
 auto_set_kconfig() {
-	local files=$(git diff --diff-filter=ACMR --name-only $base_sha..$head_sha)
+	local files=$(git diff --diff-filter=ACM --no-renames --name-only $base_sha..$head_sha)
 	declare -a o_files
 
 	echo "get_kconfig on range $base_sha..$head_sha"