[zypp-devel] Is it possible to strip trailing whitespace on svn commit?
We sometimes create large diffs, just because some editors leave trailing whitespace garbage in the source files, and some don't. Can't we strip trailing whitespace on commit? I guess svn has some sophisticated-hook for this? -- cu, Michael Andres +------------------------------------------------------------------+ Key fingerprint = 2DFA 5D73 18B1 E7EF A862 27AC 3FB8 9E3A 27C6 B0E4 +------------------------------------------------------------------+ Michael Andres YaST Development ma@novell.com SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nuernberg) Maxfeldstrasse 5, D-90409 Nuernberg, Germany, ++49 (0)911 - 740 53-0 +------------------------------------------------------------------+ -- To unsubscribe, e-mail: zypp-devel+unsubscribe@opensuse.org For additional commands, e-mail: zypp-devel+help@opensuse.org
Hi, On Wed, 6 Aug 2008, Michael Andres wrote:
We sometimes create large diffs, just because some editors leave trailing whitespace garbage in the source files, and some don't.
Can't we strip trailing whitespace on commit? I guess svn has some sophisticated-hook for this?
That wouldn't work. If the finally committed data is not the same data that also was send by the user an update will immediately create a merge conflict. What needs to happen is that the committer local files are without trailing whitespace before starting to commit. There are hooks for _rejecting_ commits based on content, we could misuse that. Quite unforgiving, though. Ciao, Michael. -- To unsubscribe, e-mail: zypp-devel+unsubscribe@opensuse.org For additional commands, e-mail: zypp-devel+help@opensuse.org
On Wed, Aug 06, Michael Matz wrote:
That wouldn't work. If the finally committed data is not the same data that also was send by the user an update will immediately create a merge conflict. What needs to happen is that the committer local files are without trailing whitespace before starting to commit. There are hooks for _rejecting_ commits based on content, we could misuse that. Quite
You realy think it's a missuse? ;)
unforgiving, though.
Hm. Then everybody please check whether your editor supports removal if trailing whitespace. -- cu, Michael Andres +------------------------------------------------------------------+ Key fingerprint = 2DFA 5D73 18B1 E7EF A862 27AC 3FB8 9E3A 27C6 B0E4 +------------------------------------------------------------------+ Michael Andres YaST Development ma@novell.com SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nuernberg) Maxfeldstrasse 5, D-90409 Nuernberg, Germany, ++49 (0)911 - 740 53-0 +------------------------------------------------------------------+ -- To unsubscribe, e-mail: zypp-devel+unsubscribe@opensuse.org For additional commands, e-mail: zypp-devel+help@opensuse.org
participants (2)
-
Michael Andres -
Michael Matz