On Apr 22, 2014, at 22:22, Justin Mrkva wrote: > I see, that’s good to know. I’ll definitely set it up, it does make sense to > strip whitespace. Not sure why that isn’t done by default. :) > > Now I just have to figure out if a dump/load will apply the hooks to to clean > up old log messages that have trailing whitespace. Not a big deal in the > scheme of things though.
dump/load uses svnadmin and does not invoke hook scripts. But there’s no need to dump and load to correct revision properties of past revisions. Just install a pre-commit-hook script allowing modification of those properties, then run log-police.py with the --all-revs option, as the usage message shows: USAGE: log-police.py [-t TXN_NAME | -r REV_NUM | --all-revs] REPOS Ensure that log messages end with exactly one newline and no other whitespace characters. Use as a pre-commit hook by passing '-t TXN_NAME'; fix up a single revision by passing '-r REV_NUM'; fix up all revisions by passing '--all-revs'. (When used as a pre-commit hook, may modify the svn:log property on the txn.)
