On Tue, May 26, 2009 at 11:41:17AM +0100, Trevor Daniels wrote:
I've looked again more closely at your original patch. The
reason
it fails to apply is that line 45 in the existing version of
/input/lsr/non-default-tuplet-numbers.ly has no trailing space
(actually
it's just a single blank line), but your patch attempts to match
it with
a line containing a single space. The same is true of line 13 in
/input/new/non-default-tuplet-numbers.ly. There may be more; I
think git am reports just the first one in each file.
Sweet mao, this is getting beyond ridiculous.
FROGS: for the love of free love and open source hippydom,
somebody make a bloody python script that removes tailing spaces
from lines. Then run it on our docs, including input/ stuff.
Then add it to scripts/auxiliar, and make makelsr.py call it when
it's finished, so we'll always have trailing-space-less input/lsr/
files.
Expected time: 10 minutes (60 for a complete python newbie) for
the initial python script, 10 minutes for running it, verifying
the diffs look right, etc. 30 minutes for adding it to git,
modifying makelsr.py, etc. Total 50 (or 100) minutes.
Plus another 50-100 minutes for dealing with trailing whitespaces
in the python script itself.