Re: *.profile file changes in regression tests?

2011-11-03 Thread Graham Percival
On Thu, Nov 03, 2011 at 12:38:02AM +, Adam Spiers wrote: Do I need to create an issue and/or Rietveld entry for this and for my other patch to {doc,cg}-section.sh, or will someone who knows the processes (e.g. Bug Squad member) take care of this? Ideally, upload it with our git-cl:

Re: multiple issues with building docs (patch included)

2011-11-03 Thread Graham Percival
On Wed, Nov 02, 2011 at 09:13:02PM +0100, David Kastrup wrote: Graham Percival gra...@percival-music.ca writes: At this very moment, - we cannot build releases Well, they _are_ development releases anyway. While my computing facilities are quite limited (single Pentium M of moderate

Re: multiple issues with building docs (patch included)

2011-11-03 Thread David Kastrup
Graham Percival gra...@percival-music.ca writes: On Wed, Nov 02, 2011 at 09:13:02PM +0100, David Kastrup wrote: Graham Percival gra...@percival-music.ca writes: At this very moment, - we cannot build releases Well, they _are_ development releases anyway. While my computing facilities

Re: multiple issues with building docs (patch included)

2011-11-03 Thread Graham Percival
On Thu, Nov 03, 2011 at 08:40:46AM +0100, David Kastrup wrote: And as long as nobody commits directly to master, dev/staging does not ~ what if Han-Wen or John pops in to make a quick bugfix (say, to the translation handling stuff), hasn't spent

Re: multiple issues with building docs (patch included)

2011-11-03 Thread Graham Percival
On Wed, Nov 02, 2011 at 09:13:02PM +0100, David Kastrup wrote: Graham Percival gra...@percival-music.ca writes: At this very moment, - we cannot build releases Well, they _are_ development releases anyway. While my computing facilities are quite limited (single Pentium M of moderate

Re: [translations] Re: CG 5.8.3: updating committish of lsr snippets

2011-11-03 Thread Federico Bruni
Il 01/11/2011 20:32, Francisco Vila ha scritto: I do not see this output especially abnormal. As you know, from this output you will obtain one of -- changes to lilypond code. You should ignore it. -- changes to translations doctitleXX or texidocXX other than yours. Ignore those, too.

Re: multiple issues with building docs (patch included)

2011-11-03 Thread David Kastrup
Graham Percival gra...@percival-music.ca writes: On Thu, Nov 03, 2011 at 08:40:46AM +0100, David Kastrup wrote: And as long as nobody commits directly to master, dev/staging does not ~ what if Han-Wen or John pops in to make a quick bugfix

Re: Doc: NR Added new node for Custom Footnotes (issue 5315053)

2011-11-03 Thread Graham Percival
On Wed, Nov 02, 2011 at 09:34:12AM +, mts...@gmail.com wrote: So that's my reasoning. But the question why didn't Mike... should be addressed to Han-Wen: as he was the one who created it, he may have answers that I don't. ok, good answer. - Graham

Re: *.profile file changes in regression tests?

2011-11-03 Thread Graham Percival
On Thu, Nov 03, 2011 at 03:28:14AM +, Carl Sorensen wrote: I tried to apply this patch, but couldn't, probably because of the way Outlooks works (or doesn't) on OS/X. Don't forget that various mail servers automatically add a in front of any line beginning with From. That breaks every

Re: multiple issues with building docs (patch included)

2011-11-03 Thread David Kastrup
Graham Percival gra...@percival-music.ca writes: On Wed, Nov 02, 2011 at 09:13:02PM +0100, David Kastrup wrote: Graham Percival gra...@percival-music.ca writes: At this very moment, - we cannot build releases Well, they _are_ development releases anyway. While my computing facilities

Re: multiple issues with building docs (patch included)

2011-11-03 Thread Trevor Daniels
David Kastrup wrote Thursday, November 03, 2011 9:01 AM Graham Percival gra...@percival-music.ca writes: and there's no point having everbody work on dev/staging instead of master; we'll still get random breakage and so we'll be no better off than having everybody working on master.

Re: [translations] Re: CG 5.8.3: updating committish of lsr snippets

2011-11-03 Thread Francisco Vila
2011/11/3 Federico Bruni fedel...@gmail.com: On the other hand, I thought (and still think) that the right committish, which keeps tracking information AND gives a terse check-translation output, should exist for these files. Not necesarily, because snippets change. That is where garbage comes

Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread reinhold . kainhofer
LGTM, although regtests are missing (for \once applied to multiple settings at one, stored in a variable). http://codereview.appspot.com/5322065/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Peekay Ex
Adam, On Wed, Nov 2, 2011 at 5:58 PM, adam.spi...@gmail.com wrote: I think I am now finished.  The new patch series is available at    https://github.com/aspiers/lilypond/commits/jazz IMHO there are now too many patches in the series to combine into a single commit for review.  To do so

Re: Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread dak
Reviewers: Reinhold, Message: On 2011/11/03 10:47:59, Reinhold wrote: LGTM, although regtests are missing (for \once applied to multiple settings at one, stored in a variable). There you are. Description: Implement \once as music function able to operate on complex stuff. Please review

Re: Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread dak
On 2011/11/03 11:45:00, Reinhold wrote: http://codereview.appspot.com/5322065/diff/3001/input/regression/complex-once.ly File input/regression/complex-once.ly (right): http://codereview.appspot.com/5322065/diff/3001/input/regression/complex-once.ly#newcode11

Re: changing-defaults.itely: correct misstatement about variables for context mods (issue 5306076)

2011-11-03 Thread dak
http://codereview.appspot.com/5306076/diff/3002/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (left): http://codereview.appspot.com/5306076/diff/3002/Documentation/notation/changing-defaults.itely#oldcode732

Re: Error when compiling with 2.15.16 for \drums function

2011-11-03 Thread Peekay Ex
Mikem On Wed, Nov 2, 2011 at 4:55 PM, m...@apollinemike.com m...@apollinemike.com wrote: On Nov 2, 2011, at 5:41 PM, Peekay Ex wrote: Hello, While helping someone on the lists yesterday I noticed that a file would not compile on 2.15.16 but was ok on 2.14.2. Because of the error message

Re: changing-defaults.itely: correct misstatement about variables for context mods (issue 5306076)

2011-11-03 Thread pkx166h
http://codereview.appspot.com/5306076/diff/3002/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (left): http://codereview.appspot.com/5306076/diff/3002/Documentation/notation/changing-defaults.itely#oldcode732

Re: changing-defaults.itely: correct misstatement about variables for context mods (issue 5306076)

2011-11-03 Thread dak
On 2011/11/03 13:15:23, J_lowe wrote: http://codereview.appspot.com/5306076/diff/3002/Documentation/notation/changing-defaults.itely File Documentation/notation/changing-defaults.itely (left):

Re: Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread dak
As an illustration: take a look at ly/property-init.ly. It contains dozens of commands that will cooperate with the new \once. All of them (well, the reverting commands don't, just because \once\revert is not [yet?] workable in general). That's its principal selling point. And I don't want to

Re: Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread n . puttock
LGTM. http://codereview.appspot.com/5322065/diff/3001/input/regression/complex-once.ly File input/regression/complex-once.ly (right): http://codereview.appspot.com/5322065/diff/3001/input/regression/complex-once.ly#newcode7 input/regression/complex-once.ly:7: \layout { ragged-right = ##t }

Re: Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread dak
http://codereview.appspot.com/5322065/diff/3001/input/regression/complex-once.ly File input/regression/complex-once.ly (right): http://codereview.appspot.com/5322065/diff/3001/input/regression/complex-once.ly#newcode7 input/regression/complex-once.ly:7: \layout { ragged-right = ##t } On

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Adam Spiers
On Thu, Nov 03, 2011 at 11:13:28AM +, Peekay Ex wrote: Adam, On Wed, Nov 2, 2011 at 5:58 PM, adam.spi...@gmail.com wrote: I think I am now finished.  The new patch series is available at    https://github.com/aspiers/lilypond/commits/jazz IMHO there are now too many patches in

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Graham Percival
On Thu, Nov 03, 2011 at 04:00:41PM +, Adam Spiers wrote: On Thu, Nov 03, 2011 at 11:13:28AM +, Peekay Ex wrote: One patch per tracker item? I can do that if noone objects to tracker items for patches as trivial as converting tabs to whitespace? I'd rather not see those patches.

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Adam Spiers
On Thu, Nov 03, 2011 at 04:28:53PM +, Graham Percival wrote: On Thu, Nov 03, 2011 at 04:00:41PM +, Adam Spiers wrote: On Thu, Nov 03, 2011 at 11:13:28AM +, Peekay Ex wrote: One patch per tracker item? I can do that if noone objects to tracker items for patches as trivial

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Adam Spiers
On Thu, Nov 3, 2011 at 4:28 PM, Graham Percival gra...@percival-music.ca wrote: We're not comfortable with git.  Other than 4 or 5 people, each person who's started pushing to dev/staging has required between 3 and 10 emails to get them able to reliably push to a branch without screwing stuff

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Carl Sorensen
On 11/3/11 10:50 AM, Adam Spiers lilypond-de...@adamspiers.org wrote: On Thu, Nov 03, 2011 at 04:28:53PM +, Graham Percival wrote: On Thu, Nov 03, 2011 at 04:00:41PM +, Adam Spiers wrote: On Thu, Nov 03, 2011 at 11:13:28AM +, Peekay Ex wrote: One patch per tracker item? I

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Graham Percival
On Thu, Nov 03, 2011 at 06:28:55PM +, Carl Sorensen wrote: On 11/3/11 10:50 AM, Adam Spiers lilypond-de...@adamspiers.org wrote: and given my strong aversion to (a) a mix of indentation styles within a single file, and (b) commits which mix whitespace changes with real coding changes, I

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread David Kastrup
Graham Percival gra...@percival-music.ca writes: On Thu, Nov 03, 2011 at 06:28:55PM +, Carl Sorensen wrote: On 11/3/11 10:50 AM, Adam Spiers lilypond-de...@adamspiers.org wrote: and given my strong aversion to (a) a mix of indentation styles within a single file, and (b) commits which

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Carl Sorensen
On 11/3/11 1:13 PM, David Kastrup d...@gnu.org wrote: Graham Percival gra...@percival-music.ca writes: My understanding is that Jan insisted on the script matching emacs' formatting, but as long as that's done he won't object. (come to think of it, that may have been private email after

Re: Fix issue 11 -- beamlet points in wrong direction on tuplet (issue 4941041)

2011-11-03 Thread pkx166h
Passes make and make check gives one reg test diff attached on http://code.google.com/p/lilypond/issues/detail?id=11#c20 James http://codereview.appspot.com/4941041/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Patch series (available at branch dev/syntax) for more argument list power. (issue 5333051)

2011-11-03 Thread pkx166h
Passes make and make check, No reg test diffs. James http://codereview.appspot.com/5333051/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Issue 1954: Implement event type filtering for pointAndClick events. (issue 5327061)

2011-11-03 Thread pkx166h
Passes make and no reg test diffs. James http://codereview.appspot.com/5327061/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Fixes slope errors from incorrect X extents in Beam::print. (issue 5293060)

2011-11-03 Thread pkx166h
Passes Make but a lot of reg test diffs show up Load index.html from the zip here: http://lilypond-stuff.1065243.n5.nabble.com/Tracker-1986-reg-tests-td4962636.html James http://codereview.appspot.com/5293060/ ___ lilypond-devel mailing list

Re: Fixes issue 2005 (too much script padding between certain scripts and accidental) (issue 5328063)

2011-11-03 Thread pkx166h
Passes Make but no reg test diffs show. James http://codereview.appspot.com/5328063/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Fixes NoteColumn vs SpanBar collisions. (issue 5323062)

2011-11-03 Thread pkx166h
Passes Make but lots of reg test diffs here: http://lilypond-stuff.1065243.n5.nabble.com/Tracker-issue-2000-reg-test-diffs-td4962666.html James http://codereview.appspot.com/5323062/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Fixes to jazz chord displays (issue 5320074)

2011-11-03 Thread Carl . D . Sorensen
Reviewers: , Message: I've uploaded Adam's changes for review. I removed the spacing changes in scm/chord-ignatzek-names.scm awaiting resolution on .scm indentation. Description: Fixes to jazz chord displays This issue consists of a well-formed series of commits created by Adam Spiers. The

Re: changes in chord names formatting (1503, 1572) (issue 4981052)

2011-11-03 Thread Carl Sorensen
On 11/3/11 10:50 AM, Adam Spiers lilypond-de...@adamspiers.org wrote: On Thu, Nov 03, 2011 at 04:28:53PM +, Graham Percival wrote: On Thu, Nov 03, 2011 at 04:00:41PM +, Adam Spiers wrote: On Thu, Nov 03, 2011 at 11:13:28AM +, Peekay Ex wrote: Moving back to the jazz patches:

Re: Implement \once as music function able to operate on complex stuff. (issue 5322065)

2011-11-03 Thread pkx166h
Passes make and no reg test diffs. James http://codereview.appspot.com/5322065/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Fixes to jazz chord displays (issue 5320074)

2011-11-03 Thread pkx166h
Patch doesn't apply to current master.. --snip-- jlowe@jlowe-lilybuntu2:~/lilypond-git$ patch -p1 ../Desktop/issue5320074_2001.diff patching file Documentation/included/chord-names-jazz.ly patching file Documentation/notation/chords.itely patching file

PATCH: Countdown to 20111106

2011-11-03 Thread Colin Campbell
For 20:00 MST Sunday November 6th Issue 1567 http://code.google.com/p/lilypond/issues/detail?id=1567: Add documentation for footnotes - R 5315053 http://codereview.appspot.com/5315053/ Issue 1954 http://code.google.com/p/lilypond/issues/detail?id=1954: request: filter point-and-click

Re: Fixes to jazz chord displays (issue 5320074)

2011-11-03 Thread Carl . D . Sorensen
New patch set uploaded. Rebased to current master. http://codereview.appspot.com/5320074/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel