tools: Improve uncrustify filtering algorithm (#21)

Instead of deiniting git submodules, use `git submodule status` to find
the list of exclude directories.

Signed-off-by: Devaraj Ranganna <[email protected]>

Tools/scripts/run_uncrustify.sh

@@ -4,6 +4,21 @@
 # <[email protected]>
 # SPDX-License-Identifier: MIT
 
-git submodule deinit --all -f
-find . -iname "*.[hc]" -exec uncrustify --replace --no-backup --if-changed -c Tools/uncrustify.cfg -l C {} +
-git submodule update --init --recursive
+# Ensure the new line is used to split the output of the `git submodule status`
+# command into an array
+IFS=$'\n'
+
+submodules_status=($(git submodule status))
+
+exclude_pattern="-E "
+for submodule_status in "${submodules_status[@]}"
+do
+    # Ensure the space is used to split the submodule status into an array
+    IFS=" "
+    submodule_status_parts=($submodule_status)
+    submodule_path = "${submodule_status_parts[1]}"
+    exclude_pattern+="${submodule_path} -E "
+done
+exclude_pattern+="./build"
+
+fdfind -E $exclude_pattern -e c -e h --exec uncrustify --no-backup --replace --if-changed -c Tools/uncrustify.cfg -l C

ci/pipeline-baseline-fri.yml

@@ -48,6 +48,8 @@ pre-commit:
   stage: quality-check
   script:
     - pip install . -t $PWD
+    - apt-get update -y
+    - apt-get install fd-find -y
     - cp $PWD/bin/banned-api-hook $PWD/banned-api-hook
     - export PATH="$PWD:$PATH";
     - pre-commit run --all-files --verbose --show-diff-on-failure

release_changes/202311221345.change

@@ -0,0 +1 @@
+tools: Improve `uncrustify` filtering algorithm