Skip to content
Snippets Groups Projects
Commit 8bf24f51 authored by Gerald Combs's avatar Gerald Combs Committed by Alexis La Goutte
Browse files

Pre-commit: Don't force whitespace checks everywhere.

Move the whitespace check (git diff-index --check --cached) inside the
CHECK_FILES loop, otherwise editing SVGs with Inkscape will be much less
convenient.

Change-Id: I2b9e3575d54091572caa89e3e317e5e121984010
Reviewed-on: https://code.wireshark.org/review/25144


Reviewed-by: default avatarGerald Combs <gerald@wireshark.org>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot
Reviewed-by: default avatarAlexis La Goutte <alexis.lagoutte@gmail.com>
parent d239da26
Branches
Tags
No related merge requests found
......@@ -90,6 +90,9 @@ for FILE in $CHECK_FILES; do
#Check if checkfiltername is good
./tools/checkfiltername.pl $FILE || exit_status=1
# If there are whitespace errors, print the offending file names and fail. (from git pre-commit.sample)
git diff-index --check --cached ${COMMIT_ID} $FILE || exit_status=1
done
if [ "x$DIAMETER_FILES" != x ]
......@@ -97,9 +100,6 @@ then
./tools/validate-diameter-xml.sh > /dev/null || exit_status=1
fi
# If there are whitespace errors, print the offending file names and fail. (from git pre-commit.sample)
git diff-index --check --cached ${COMMIT_ID} || exit_status=1
exit $exit_status
#
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment