PATCHES: Countdown for October 4th

2016-10-04 Thread James Lowe
Hello,

Here is the current patch countdown list. The next countdown will be on
October 7th.

A quick synopsis of all patches currently in the review process can be
found here:

http://Philholmes.net/lilypond/allura/


__



Push:


4974 Add output-attributes grob property to replace id - Paul Morris
https://sourceforge.net/p/testlilyissues/issues/4974
http://codereview.appspot.com/308430043


Countdown:


4979 Pitched trill: ledger lines are connected - Paul Morris
https://sourceforge.net/p/testlilyissues/issues/4979
http://codereview.appspot.com/308560043


Review:


4980 Fix issue where Musicxml2ly generates improper code for minor
chords - James Lowe
https://sourceforge.net/p/testlilyissues/issues/4980
http://codereview.appspot.com/305700043


4937 [GSoC] Implement cross-voice dynamic spanners - Nathan Chou
https://sourceforge.net/p/testlilyissues/issues/4937
http://codereview.appspot.com/304160043


New: No New patches at this time.



Waiting:


4600 Let notes/rests suppress multi-measure rest grobs - Dan Eble
https://sourceforge.net/p/testlilyissues/issues/4600
http://codereview.appspot.com/265160043



Regards


James

___

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Fixing "id.ly" regtest file

2016-10-04 Thread Paul

Hi all,

I pushed my commits for issue 4974 to staging, and all went well, but 
then I noticed that the regtest "id.ly" was incorrect.  The diff for it has:


-{ \override NoteHead.id = #"foo" c }
+{ \override NoteHead.output-attributes.id = #"foo" c }

when this shouldn't have been changed.  (Running the 
scripts/auxiliar/...  convert-ly script made this change but I 
overlooked it because the header text in that file was also changed 
manually.  So I didn't manually correct it.)


To fix this, I'd propose reverting the last commit on master:
http://git.savannah.gnu.org/cgit/lilypond.git/commit/?id=03d47798d39bd9ce2a87376ea1585f1f31c1b81b

and then I can re-upload an amended version of that commit that fixes 
the "id.ly" file.  Please let me know how best to proceed.


Sorry for not catching this before,
-Paul
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel