Hello -----Original Message----- From: Graham Percival <gra...@percival-music.ca> Date: Sat, 22 Jan 2011 20:59:11 +0000 To: "c_soren...@byu.edu" <c_soren...@byu.edu> Cc: James Lowe <james.l...@datacore.com>, lilypond-devel <lilypond-devel@gnu.org> Subject: Re: Issue 1228: 5-month-old PATCH
>On Sat, Jan 22, 2011 at 01:51:26PM -0700, Carl Sorensen wrote: >> On 1/22/11 1:32 PM, "James Lowe" <james.l...@datacore.com> wrote: >> >> > I was just going to add it as a *.ly file in the ly/ dir. Then was >>going >> > to ask how to add the \function to files in the source so that it >>could be >> > called. >> >> Well, I'm sorry that I overwhelmed you with my answer. We could do it >>with >> a .ly file. > >As a (possibly temporary) solution -- or even just as a "first >draft of a patch" -- how about this: >1. add a file containing the scheme functions. Call it >ly/merge-functions.ly >2. add the property definitions to ly/property-init.ly >(the mergeRestsOn, mergeRestsOff, mergeRests stuff) >3. add \include merge-functions.ly to propert-init.ly >4. send patch for review+comment. Added as http://codereview.appspot.com/4005046/ I did step 3 slightly differently as it seemed all the \includes were in declarations-init.ly, I checked in there and it seemed the obvious place. I have done a make ; make doc and had no errors. I'll also announce the patch 'properly' on Dev James _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel