Created
June 16, 2011 06:38
-
-
Save mrkschan/1028787 to your computer and use it in GitHub Desktop.
a pre-commit hook for git to remove trailing whitespace
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#!/bin/sh | |
# | |
# A git hook script to find and fix trailing whitespace | |
# in your commits. Bypass it with the --no-verify option | |
# to git-commit | |
# Ref - http://is.gd/PerowD | |
# | |
if git rev-parse --verify HEAD >/dev/null 2>&1 | |
then | |
against=HEAD | |
else | |
# Initial commit: diff against an empty tree object | |
against=4b825dc642cb6eb9a060e54bf8d69288fbee4904 | |
fi | |
# Find files with trailing whitespace | |
for FILE in `exec git diff-index --check $against -- | sed '/^[+-]/d' | (sed -r 's/:[0-9]+:.*//' > /dev/null 2>&1 || sed -E 's/:[0-9]+:.*//') | uniq` ; do | |
# Fix them! | |
(sed -i 's/[[:space:]]*$//' "$FILE" > /dev/null 2>&1 || sed -i '' -E 's/[[:space:]]*$//' "$FILE") | |
git add "$FILE" | |
echo "NOTE: removed trailing whitespace from $FILE" | |
done | |
exit |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
(sed -r 's/:[0-9]+:.*//' > /dev/null 2>&1 || sed -E 's/:[0-9]+:.*//')
^ is this for PC / Mac compatibility? It's broken on Linux. The first sed will match, the output will be sent to /dev/null, and there's nothing left to iterate on.
better setting up a $SED environment variable containing 'sed -r' or 'sed -E' upfront if you insist on out-of-the box compatibility.
Great stuff though, thanks a lot for sharing!