mirror/OpenMW
1
0
Fork 0
mirror of https://gitlab.com/OpenMW/openmw.git synced 2025-01-17 01:10:10 +00:00
Code Issues Projects Releases Wiki Activity

Use clang-format --dry-run instead of relying on git diff

Browse Source
This commit is contained in:
ζeh Matt 2022-09-22 21:45:47 +03:00
parent ddb0522bbf
commit 45bfed3b42
No known key found for this signature in database
GPG Key ID: 18CE582C71A225B0
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
CI/check_clang_format.sh
View File

@ -5,19 +5,23 @@ HAS_DIFFS=0
check_format() { check_format() {
local path=$1 local path=$1
local tempfile=$(mktemp)
for item in $(find $path -type f -name "*"); for item in $(find $path -type f -name "*");
do do
if [[ "$item" =~ .*\.(cpp|hpp|h) ]]; then if [[ "$item" =~ .*\.(cpp|hpp|h) ]]; then
echo "Checking code formatting on $item" echo "Checking code formatting on $item"
$CLANG_FORMAT $item > $tempfile $CLANG_FORMAT --dry-run -Werror "$item"
git diff --color=always --no-index $item $tempfile
if [[ $? = 1 ]]; then if [[ $? = 1 ]]; then
local tempfile=$(mktemp)
# Avoid having different modes in the diff.
chmod --reference="$item" "$tempfile"
# Generate diff
$CLANG_FORMAT "$item" > "$tempfile"
git diff --color=always --no-index $item $tempfile
rm -f "$tempfile"
HAS_DIFFS=1 HAS_DIFFS=1
fi fi
fi; fi;
done; done;
rm -f $tempfile
} }
check_format "./apps" check_format "./apps"