Comments on snippet: 'Git whitespace fixes for windows newlines'

benpjohnson said on 21/Sep/2008

Sun, 21 Sep 2008 14:44:27 UTC

Doesn't really work for most issues had to disable it in the pre-commit hook file for time being

Mikelifeguard said on 17/Feb/2010

Wed, 17 Feb 2010 23:14:46 UTC

One of the seds seems to be wrong. Try: # Find files with trailing whitespace for FILE in `exec git diff-index --check --cached --ignore-space-at-eol $against -- | sed '/^[+-]/d' | sed -r 's/:[0-9]+:.*//' | uniq`; do # Fix them! sed -i 's/[[:space:]]+$//' $FILE done

Mikelifeguard said on 17/Feb/2010

Wed, 17 Feb 2010 23:19:20 UTC

HAH, remove that `--ignore-space-at-eol` obviously. That's *exactly* what we're trying to find `:D`

Mikelifeguard said on 17/Feb/2010

Wed, 17 Feb 2010 23:32:45 UTC

Sorry, again! One of my seds has a typo too! This time I've actually copy-pasted from my pre-commit hook instead of typing it out. Note that `$c` is already found further up in the script. This is $against in my broken example above, and is in the .sample script already - I just shortened it. for f in `exec git diff-index --cached --check $c -- | sed '/^[-+]/d' | sed -r 's:[0-9]+:.*//' | uniq` ; do sed -i 's/[[:space:]]*$//' $f done