PATCH: 48-hour countdown

2011-07-13 Thread Colin Campbell
22:00 MST Friday July 15 Issue 1390 : Tied acciaccaturas should not be printed with a slur - Reitveld issue 4670056 Issue 1760 : \RemoveE

Re: Adapt fixcc.py to use Astyle and/or emacs (issue4662074)

2011-07-13 Thread k-ohara5a5a
It seems everyone is happy to use astyle as the indenter (it sure is faster) so here is one last patch set showing what this does to the sample set of C++ files. Running fixcc(astyle) on the source tree again gives a clean make check. http://codereview.appspot.com/4662074/diff/26001/Documentati

Re: an example of minimal example (issue4636082)

2011-07-13 Thread ColinPKCampbell
This has had its countdown; please push and close. http://codereview.appspot.com/4636082/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: correcting note about rerunning regtests (issue4675048)

2011-07-13 Thread ColinPKCampbell
This has had its countdown; please push and close. http://codereview.appspot.com/4675048/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Adds redirect-lilypond-output option tolilypond-book(issue4664060)

2011-07-13 Thread Graham Percival
On Wed, Jul 13, 2011 at 11:04:18PM +0100, Trevor Daniels wrote: > > Graham Percival wrote Wednesday, July 13, 2011 5:57 PM > > >On Wed, Jul 13, 2011 at 03:26:18PM +0100, Phil Holmes wrote: > > >That "worst that happens" is beyond what's acceptable. Seeing an > >error message before the command

Add hihat halfopen glyph to font (issue4714043)

2011-07-13 Thread Carl . D . Sorensen
Reviewers: , Message: Here is the patch for the new hihat halfopen glyph as requested in Issue 1688. If you are not building in a clean directory, you will need to do rm mf/out/* && make in order to generate the font with the new glyph Description: Add hihat halfopen glyph to font Please rev

2.14.2 release

2011-07-13 Thread Carl Sorensen
Graham, Could you prepare a 2.14.2 release? I've verified that I get a clean make and make doc with stable/2.14. All of the bugfixes that can be backported have been backported, as far as I can see. I've applied all the translation work that Francisco asked for. Thanks, Carl ___

Re: Adds redirect-lilypond-output option tolilypond-book(issue4664060)

2011-07-13 Thread Trevor Daniels
Graham Percival wrote Wednesday, July 13, 2011 5:57 PM On Wed, Jul 13, 2011 at 03:26:18PM +0100, Phil Holmes wrote: My understanding (which is certainly not perfect) is that one of the stream is buffered (stdout from memory, without checking) and the other isn't, so that the worst that hap

Fix #1695: Clef change placed outside score. (issue4683043)

2011-07-13 Thread n . puttock
Reviewers: , Message: A MetronomeMark represents a special case for a break-alignable grob, since sometimes it will be aligned on noteheads rather than prefatory material. In this case, we don't want it to be acknowledged by the Break_alignment_engraver, since it will hijack the X-parent before

Re: Adds redirect-lilypond-output option to lilypond-book(issue4664060)

2011-07-13 Thread Werner LEMBERG
>> >Redirecting two file pointers to the same file has a tendency to >> >wreak havoc with buffering. So it is not something I am overly >> >comfortable with. > > ok, if/when we add this, we'd use a different way of recording > output. I'm certain that python's subprocess module has some way of

Re: Allows for rider grobs in outside-staff-priority. (issue4639075)

2011-07-13 Thread Han-Wen Nienhuys
On Wed, Jul 13, 2011 at 5:12 PM, m...@apollinemike.com wrote: > On Jul 13, 2011, at 4:42 PM, Han-Wen Nienhuys wrote: > >> On Wed, Jul 13, 2011 at 11:27 AM, m...@apollinemike.com >> wrote: >> Wouldnt it be much easier for the tuplet number's extents  to be ignored for skyline purposes if

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread m...@apollinemike.com
On Jul 13, 2011, at 10:09 PM, Neil Puttock wrote: > On 13 July 2011 21:03, Neil Puttock wrote: >> On 13 July 2011 20:51, m...@apollinemike.com wrote: >> >>> Was I too narrow in my original fix? I can work on something cleaner to >>> address the segfault that doesn't lead to cyclical dependenc

Re: Allows for rider grobs in outside-staff-priority. (issue4639075)

2011-07-13 Thread m...@apollinemike.com
On Jul 13, 2011, at 4:42 PM, Han-Wen Nienhuys wrote: > On Wed, Jul 13, 2011 at 11:27 AM, m...@apollinemike.com > wrote: > >>> Wouldnt it be much easier for the tuplet number's extents to be >>> ignored for skyline purposes if there already is an associated tuplet >>> bracket? >>> >> >> That's

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread Neil Puttock
On 13 July 2011 21:03, Neil Puttock wrote: > On 13 July 2011 20:51, m...@apollinemike.com wrote: > >> Was I too narrow in my original fix?  I can work on something cleaner to >> address the segfault that doesn't lead to cyclical dependencies so that this >> doesn't perturb future generations of

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread Neil Puttock
On 13 July 2011 20:51, m...@apollinemike.com wrote: > Was I too narrow in my original fix?  I can work on something cleaner to > address the segfault that doesn't lead to cyclical dependencies so that this > doesn't perturb future generations of regtests. That would be great. BTW, like James,

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread m...@apollinemike.com
On Jul 13, 2011, at 9:37 PM, Neil Puttock wrote: > On 13 July 2011 11:22, m...@apollinemike.com wrote: > >> You're right - they would have been the same. I think this error may have >> been introduced in a recent patch, in which case it is a regression. I'd >> have to run git-bisect to figur

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread Neil Puttock
On 13 July 2011 11:22, m...@apollinemike.com wrote: > You're right - they would have been the same.  I think this error may have > been introduced in a recent patch, in which case it is a regression.  I'd > have to run git-bisect to figure it out, though, but all of my CPU is being > eaten up

Re: Adds redirect-lilypond-output option to lilypond-book (issue4664060)

2011-07-13 Thread Graham Percival
On Tue, Jul 12, 2011 at 03:56:59PM +0100, Phil Holmes wrote: > - Original Message - From: > >in light of the growing consensus for "combined" logfiles for the build > >system -- and given that lilypond only produces stuff on stderr and > >apparently this isn't going to change -- I wonder i

Re: Adds redirect-lilypond-output option to lilypond-book(issue4664060)

2011-07-13 Thread Graham Percival
On Wed, Jul 13, 2011 at 03:26:18PM +0100, Phil Holmes wrote: > - Original Message - From: "David Kastrup" > To: > >percival.music...@gmail.com writes: > > > >>in light of the growing consensus for "combined" logfiles for the build > >>system -- and given that lilypond only produces stuff

Re: Fix #1507: Inconsistent \festival output in regression testing. (issue4693046)

2011-07-13 Thread Graham Percival
On Wed, Jul 13, 2011 at 06:18:30AM +0200, Werner LEMBERG wrote: > > reset *skip-word* to default at end of file to prevent override > > leaking into other regression tests > > Just curious: How can this happen? While handling the regression > suite, more than a single test file is processed b

Re: Make test-baseline fails

2011-07-13 Thread Graham Percival
On Wed, Jul 13, 2011 at 03:01:29PM +, James Lowe wrote: > > ) Remember that our build system cannot reliably handle a build > )directory with anything inside it. > > Yes, hence my reason not using it. > > Anyway, I did a load of make clean/doc clean and re-ran autogen.sh and > configure and

RE: Make test-baseline fails

2011-07-13 Thread James Lowe
Hello, )-Original Message- )From: Graham Percival [mailto:gra...@percival-music.ca] )Sent: 12 July 2011 21:48 )To: James Lowe )Cc: lilypond-devel (lilypond-devel@gnu.org) )Subject: Re: Make test-baseline fails ) )On Tue, Jul 12, 2011 at 07:39:20PM +, James Lowe wrote: )> I'v done this

Re: Allows for rider grobs in outside-staff-priority. (issue4639075)

2011-07-13 Thread Han-Wen Nienhuys
On Wed, Jul 13, 2011 at 11:27 AM, m...@apollinemike.com wrote: >> Wouldnt it be much easier for the tuplet number's extents  to be >> ignored for skyline purposes if there already is an associated tuplet >> bracket? >> > > That's what the new line 250 of axis-group-interface.cc in the patch does:

Re: Allows for rider grobs in outside-staff-priority. (issue4639075)

2011-07-13 Thread m...@apollinemike.com
On Jul 13, 2011, at 4:13 PM, Han-Wen Nienhuys wrote: > On Fri, Jul 1, 2011 at 1:09 PM, Han-Wen Nienhuys wrote: >>> The problem with this is that, in the axis-group-interface, the >>> TupletNumber either: >>> >>> (1) Does not have an outside staff priority and therefore is included in >>> the s

Re: Adds redirect-lilypond-output option to lilypond-book(issue4664060)

2011-07-13 Thread Phil Holmes
- Original Message - From: "David Kastrup" To: Sent: Wednesday, July 13, 2011 9:59 AM Subject: Re: Adds redirect-lilypond-output option to lilypond-book(issue4664060) percival.music...@gmail.com writes: in light of the growing consensus for "combined" logfiles for the build syste

Re: Allows for rider grobs in outside-staff-priority. (issue4639075)

2011-07-13 Thread Han-Wen Nienhuys
On Fri, Jul 1, 2011 at 1:09 PM, Han-Wen Nienhuys wrote: >> The problem with this is that, in the axis-group-interface, the TupletNumber >> either: >> >> (1) Does not have an outside staff priority and therefore is included in the >> staff's skyline, which pushes the bracket too high. >> (2) Does

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread Jan Warchoł
2011/7/13 James Lowe : > See > > http://code.google.com/p/lilypond/issues/detail?id=1563#c3 > > I suddenly got a reg test difference with this patch and that .ly > > James > I get very similar output with current master (446103a87933b297c209e87fec94e97fe9462ad0), except that the line break is 2 me

Re: Move \RemoveEmptyStaves to new file for context modifications (issue #1760) (issue4664076)

2011-07-13 Thread Janek Warchoł
2011/7/13 : > Because ly/declarations-init.ly contains: > > \layout { >    \include "engraver-init.ly" > } > > So the whole contents of engraver-init.ly (including the old definition > of RemoveEmptyStaves) is interpreted and defined inside a layout block > and is thus not available at top-level,

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread m...@apollinemike.com
On Jul 13, 2011, at 12:19 PM, James Lowe wrote: > Mike, > > > )-Original Message- > )From: m...@apollinemike.com [mailto:m...@apollinemike.com] > )Sent: 13 July 2011 11:14 > )To: James Lowe > )Cc: lilypond-devel@gnu.org List > )Subject: Re: Circular dependencies for beam-skip.ly > )Impor

RE: Circular dependencies for beam-skip.ly

2011-07-13 Thread James Lowe
Mike, )-Original Message- )From: m...@apollinemike.com [mailto:m...@apollinemike.com] )Sent: 13 July 2011 11:14 )To: James Lowe )Cc: lilypond-devel@gnu.org List )Subject: Re: Circular dependencies for beam-skip.ly )Importance: Low ) )On Jul 13, 2011, at 12:12 PM, James Lowe wrote: ) )> Mi

Re: Circular dependencies for beam-skip.ly

2011-07-13 Thread m...@apollinemike.com
On Jul 13, 2011, at 12:12 PM, James Lowe wrote: > Mike, > > )-Original Message- > )From: lilypond-devel-bounces+james.lowe=datacore@gnu.org > )[mailto:lilypond-devel-bounces+james.lowe=datacore@gnu.org] On > )Behalf Of m...@apollinemike.com > )Sent: 13 July 2011 07:58 > )To: lilyp

RE: Circular dependencies for beam-skip.ly

2011-07-13 Thread James Lowe
Mike, )-Original Message- )From: lilypond-devel-bounces+james.lowe=datacore@gnu.org )[mailto:lilypond-devel-bounces+james.lowe=datacore@gnu.org] On )Behalf Of m...@apollinemike.com )Sent: 13 July 2011 07:58 )To: lilypond-devel@gnu.org List )Subject: Circular dependencies for beam-s

Re: Adds redirect-lilypond-output option to lilypond-book (issue4664060)

2011-07-13 Thread David Kastrup
percival.music...@gmail.com writes: > in light of the growing consensus for "combined" logfiles for the build > system -- and given that lilypond only produces stuff on stderr and > apparently this isn't going to change -- I wonder if it might be better > to redirect both stdout and sterr to a sin

Re: Move \RemoveEmptyStaves to new file for context modifications (issue #1760) (issue4664076)

2011-07-13 Thread reinhold . kainhofer
LGTM. On 2011/07/13 07:33:39, Janek Warchol wrote: The change looks pretty straightforward, but i don't understand why simply moving RemoveEmptyStaff definition changes anything :( Because ly/declarations-init.ly contains: \layout { \include "engraver-init.ly" } So the whole contents o

Re: Move \RemoveEmptyStaves to new file for context modifications (issue #1760) (issue4664076)

2011-07-13 Thread lemniskata . bernoullego
The change looks pretty straightforward, but i don't understand why simply moving RemoveEmptyStaff definition changes anything :( cheers, Janek http://codereview.appspot.com/4664076/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists

Circular dependencies for beam-skip.ly

2011-07-13 Thread m...@apollinemike.com
Hey all, I'm getting a lot of circular dependencies for input/regression/beam-skip.ly from the current master. I don't have time to do a full git bisect, but is this standard behavior? Cheers, MS ___ lilypond-devel mailing list lilypond-devel@gnu.org