[Box Backup] Diff Optimisations now in SVN

Jonathan Morton boxbackup@fluffy.co.uk
Tue, 18 Oct 2005 14:42:17 +0100


>> Note that a side-effect of my using KATE is that there are a lot of 
>> deltas referring to blank lines, where KATE has deleted trailing 
>> whitespace characters from the entire file being edited.  I'm not 
>> sure what to do about that, aside from trying to work out the right 
>> GNU indent parameters and hoping nothing else breaks.  Perhaps 
>> someone had better go through all the other trees and delete trailing 
>> whitespace from them too.
>
> I think the best thing to do here is 'diff -uw' your changes and apply 
> that as a patch to a new copy from trunk. Otherwise this is going to 
> cause excessive and unnecessary pain. That's exactly what I did for my 
> merge above, and it worked well.

OK, I just did that (actually, made a plain patch against trunk 
alongside an --ignore-space-change version, reversed the plain one, 
then applied the other).  This seems to have corrected the problem 
cleanly, without affecting the existence blank lines in new code as -w 
would have.

> As to whether to strip trailing spaces, I prefer that. I have emacs 
> set to do that by default, but I turn it off when working on box. KATE 
> must have an option to turn it off too (I'd consider it a bug if not). 
> But either way I don't much care.

It does have the option, but there's no way to put the spaces back in 
after the fact, except by the above laborious patching method.  Since I 
greatly prefer to use KATE than other editors these days (Emacs is OK 
to use, but an utter pain to configure the way I like it), I think 
getting rid of the trailing spaces at *some* point is a good idea.

--------------------------------------------------------------
from:     Jonathan "Chromatix" Morton
mail:     chromi@chromatix.demon.co.uk
website:  http://www.chromatix.uklinux.net/
tagline:  The key to knowledge is not to rely on people to teach you it.