Updates:
        Labels: -Patch-needs_work Patch-new

Comment #7 on issue 2240 by d...@gnu.org: Patch: Don't wrap EventChord around rhythmic events by default.
http://code.google.com/p/lilypond/issues/detail?id=2240

Ok, here is another theory about your failure: you have old versions of lily/rhythmic-music-iterator.cc and lily/include/rhythmic-music-iterator.hh in your tree, and if you do
git apply patch-from-Rietveld
you get an error message that they are already there, and they don't get updated. Remove them before calling git apply.

Here is how to avoid this in future:
Always when you use git-apply, use git apply --index.  Never anything else.
Why does this help?

It means that the patch is not just applied to the working directory, but also is something git knows about _independently_.

If you afterwards to git reset --hard, git will _not_ touch files in the working it does not know about. In this case, the old versions of the rhythmic music iterator. So they stick around perpetually.

If you did git apply --index instead, git _does_ know about those files. When the patch created them, git reset --hard will remove them again.

So please check again after removing those unregistered files from your work directory manually.


_______________________________________________
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond

Reply via email to