Re: doc-addition: the full lilypond command line with options+redirections

2010-08-13 Thread Graham Percival
On Thu, Aug 12, 2010 at 03:05:38PM -0700, -Eluze wrote:
 
 The lilypond executable may be called as follows from the command line.
 
 lilypond [option]... file...[[1]file.log] [2file.txt]

1 and 2  (and, for the matter,  ) have nothing to do with
lilypond.  I will reluctantly agree to adding something like this
at the bottom of some section:

-
@subsubsection Standard shell commands

If your shell (i.e. command window) supports normal redirects,
then you might find it useful to use:

lilypond file.ly 1stdout.log 2stderr.log all.log

// check which does which
1  normal output
2  errors
  all output

Consult documentation for your shell to see if it supports these
options, or if the syntax is different.  These commands have
nothing to do with lilypond.
-


Cheers,
- Graham

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


Re: Extra check in define-event-class patch

2010-08-13 Thread Mike Solomon
On 8/12/10 7:20 PM, Trevor Daniels t.dani...@treda.co.uk wrote:

 
 Mike Solomon wrote Thursday, August 12, 2010 10:16 AM
 
I added an extra symbol check to my event class patch:
 
 http://codereview.appspot.com/1867050
 
 Also, I have a regression test that creates a spanner engraver
 from the
 scheme end using this function (said regtest could also be turned
 into a bit
 of documentation for the contributor's guide if people feel that'd
 be
 helpful - I think it would given a recent thread on frogs
 concerning
 engravers  their complexity), but I don't know if I should submit
 it as
 part of this patch set or as a new issue on Reitveld.  Lemme know
 :-)
 
 If the regression test tests the new function I'd make
 it part of the same Reitveld issue.  If your patch makes
 it to the distribution we'd want to install both together.
 
 Trevor

Done - thank you for the suggestion.  Attached is the patch series, which
can also be found on http://codereview.appspot.com/1867050 .  Thank you!

~Mike 



0003-Adds-a-regression-test-showing-ly-define-event-class.patch
Description: Binary data


0002-Symbol-checking.patch
Description: Binary data


0001-Create-ly-define-event-class.patch
Description: Binary data
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: doc-addition: the full lilypond command line withoptions+redirections

2010-08-13 Thread Trevor Daniels


Graham Percival wrote Friday, August 13, 2010 7:28 AM



On Thu, Aug 12, 2010 at 03:05:38PM -0700, -Eluze wrote:


The lilypond executable may be called as follows from the command 
line.


lilypond [option]... file...[[1]file.log] [2file.txt]


1 and 2  (and, for the matter,  ) have nothing to do with
lilypond.  I will reluctantly agree to adding something like this
at the bottom of some section:


Done

Trevor



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


Re: Add \path markup command, and use it for \eyeglasses. (issue1730044)

2010-08-13 Thread Carl . D . Sorensen

LGTM.

Carl


http://codereview.appspot.com/1730044/show

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


Re: Fix 1084. (issue1871058)

2010-08-13 Thread percival . music . ca

Looks ok here; I just did a build from scratch with it applied.  The
only potential issue is whether it works with the regtest comparison,
but we can't test that until I make another release.

Please push.  :)

http://codereview.appspot.com/1871058/show

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


Re: NR 2.1 Vocal music

2010-08-13 Thread Graham Percival
On Thu, Aug 12, 2010 at 11:13 PM, Trevor Daniels t.dani...@treda.co.uk wrote:

 Graham Percival wrote Thursday, August 12, 2010 7:38 PM

 I'd really rather that you helped out with programming (or at
 least testing+reviewing patches) and critical doc issues...

 I looked through the 12 remaining critical issues before making
 my suggestion to tackle NR 2.1.  8 are marked as started, of
 which 2 have patches being reworked by the authors.

 Of the 4 remaining, 2 are website stuff that only you can do,
 1 Carl has promised to look at and the final one I think only
 Joe can tackle.

First -- I'm happy with you working on NR 2.1, so don't take this
email as discouraging you from doing that.  I'm aiming these comments
at the mailing list in general, and some Frogs (or even not yet
frogs), not you.

I think that 1084 was only waiting for somebody to test it with a doc
build.  I've done this now, but it's not something that's hard -- it
just takes firing off a few commands and then going for dinner.
(that said, it seems like this is a bad option for people working with
a virtual machine... but any potential contributor on linux could
certainly have done this)

Second, the release can't happen immediately after the last Critical
issue is solved.  There's about 10-20 hours of work left to do after
that.  A lot of it is technical build stuff (which I wouldn't wish on
my worst enemy), but there's still a chunk that's fairly easy to
delegate.  A quick glance at CG 10.3 should serve as sufficient
evidence of our unready state.  BTW, the first thing I'd delegate is
making a list of places to announce the release.  The current list is
about 5 years old, and even *that* list has various notes like
non-existant?.

In short: there's plenty of work, for any ability level, that needs to
be done before 2.14.0.  I can always find work for minions.

Cheers,
- Graham

PS that said, I'm not claiming that anybody *should* be focused on
making a release.  If somebody just wants to work on writing good code
and doesn't care at all about official releases and the like, then of
course just continue doing that!

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


release 2.14 timeline

2010-08-13 Thread Graham Percival
We're down to 8 Critical issues.
- 1 is a website issue that's basically finished.
- 1 is a doc issue that might already be finished (after an issue split)

That leaves 6 code issues.  3 of them have patches:
- Issue 1116:Title not centered
http://code.google.com/p/lilypond/issues/detail?id=1116
- Issue 1173:LilyPond crash when using
showFirstLength/showLastLength with \compressFullBarRests
http://code.google.com/p/lilypond/issues/detail?id=1173
- Issue 1198:Display method for \time broken
http://code.google.com/p/lilypond/issues/detail?id=1198

1 has a new programmer actively working on it:
- Issue 1096:ly:parser-parse-string segfaults when directly called
at toplevel
http://code.google.com/p/lilypond/issues/detail?id=1096
(don't steal this issue from him)

1 hasn't been touched since Feb:
- Issue 818: Bus error when adding Instrument_name_engraver to
StaffGroup and outputting MIDI.
http://code.google.com/p/lilypond/issues/detail?id=818

1 might not be a regression at all, but we need a spacing-aware
developer to look at it:
- Issue 1208:minor regression: Some combination of
'staff-affinity for two Lyrics may cause unwanted vertical space
http://code.google.com/p/lilypond/issues/detail?id=1208


Once we're down to 3 (or fewer) Critical code issues, I'll announce an
alpha version of 2.14.0 and invite normal users to try it out.  Note
that reviewing and helping fix the existing patches would satisfy this
condition.  When that happens, I expect to see another 5-15 Critical
issues from people trying it out.  Maybe less because a lot of users
are already using 2.13... but don't be surprised (or discouraged) to
see a flood of bug reports once we officially invite people to try out
2.13.

Once we're down to 0 Critical issues, we'll start having release candidates.


Translators: at the moment, I'm cautiously predicting that 2.14.0 will
be 4 to 8 weeks away.  You will have at least 2 weeks (14 days) notice
before 2.14.0 -- each release candidate is such a warning.  Francisco,
you're in charge of sending this to the translator list, and/or
sending private emails to relevant people.

Cheers,
- Graham

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


Re: Fix 1198: Fix displayLilyMusic for \time and #(set-time-signature) (issue1986042)

2010-08-13 Thread n . puttock

Hi Carl,

LGTM.

Cheers,
Neil


http://codereview.appspot.com/1986042/diff/1/2
File scm/define-music-display-methods.scm (right):

http://codereview.appspot.com/1986042/diff/1/2#newcode887
scm/define-music-display-methods.scm:887: (let* ((arguments
(ly:music-property expr 'time-signature-arguments))
I'd prefer separate properties since it would make the output of
\displayMusic less cryptic.

There's already 'numerator and 'denominator (which are currently used
for \times, even though their docstrings mention \time).

http://codereview.appspot.com/1986042/diff/1/2#newcode897
scm/define-music-display-methods.scm:897: #(set-time-signature ~a ~a
~a)~a
#(set-time-signature ~a ~a '~a)~a

http://codereview.appspot.com/1986042/diff/1/4
File scm/define-music-types.scm (right):

http://codereview.appspot.com/1986042/diff/1/4#newcode53
scm/define-music-types.scm:53: (define (make-time-signature-set music)
We should probably think about moving these to a separate file now
there's more than one callback.

(naturally this can wait for another patch)

http://codereview.appspot.com/1986042/diff/1/5
File scm/music-functions.scm (left):

http://codereview.appspot.com/1986042/diff/1/5#oldcode507
scm/music-functions.scm:507: (define-public
(make-beam-rule-time-signature-set num den rest)
Do you want to keep this?

http://codereview.appspot.com/1986042/show

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


Re: Add independent control of thickness and offset for underline markup (issue1347041)

2010-08-13 Thread n . puttock

On 2010/08/13 03:15:48, Carl wrote:


Is the default OK now?


Looks fine to me.

Cheers,
Neil

http://codereview.appspot.com/1347041/show

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


Re: Add \path markup command, and use it for \eyeglasses. (issue1730044)

2010-08-13 Thread pnorcks

On 2010/08/13 11:04:00, Carl wrote:

LGTM.



Carl


Thanks Carl.

I'll push this patchset shortly (with regression tests).

-Patrick

http://codereview.appspot.com/1730044/show

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


Re: Fix 1198: Fix displayLilyMusic for \time and #(set-time-signature) (issue1986042)

2010-08-13 Thread Carl . D . Sorensen

Let me know about the filename for the new callbacks, and I'll add that
to this patch and push it.

THanks,

Carl


http://codereview.appspot.com/1986042/diff/1/2
File scm/define-music-display-methods.scm (right):

http://codereview.appspot.com/1986042/diff/1/2#newcode887
scm/define-music-display-methods.scm:887: (let* ((arguments
(ly:music-property expr 'time-signature-arguments))
On 2010/08/13 21:25:13, Neil Puttock wrote:

I'd prefer separate properties since it would make the output of

\displayMusic

less cryptic.



There's already 'numerator and 'denominator (which are currently used

for

\times, even though their docstrings mention \time).


Ah, got it.  I hadn't thought of displayMusic.  Done.

http://codereview.appspot.com/1986042/diff/1/2#newcode897
scm/define-music-display-methods.scm:897: #(set-time-signature ~a ~a
~a)~a
On 2010/08/13 21:25:13, Neil Puttock wrote:

#(set-time-signature ~a ~a '~a)~a


Of course!  Thanks, Done.

http://codereview.appspot.com/1986042/diff/1/4
File scm/define-music-types.scm (right):

http://codereview.appspot.com/1986042/diff/1/4#newcode53
scm/define-music-types.scm:53: (define (make-time-signature-set music)
On 2010/08/13 21:25:13, Neil Puttock wrote:

We should probably think about moving these to a separate file now

there's more

than one callback.



maybe something like scm/define-music-callbacks.scm ?


(naturally this can wait for another patch)


http://codereview.appspot.com/1986042/diff/1/5
File scm/music-functions.scm (left):

http://codereview.appspot.com/1986042/diff/1/5#oldcode507
scm/music-functions.scm:507: (define-public
(make-beam-rule-time-signature-set num den rest)
On 2010/08/13 21:25:13, Neil Puttock wrote:

Do you want to keep this?


Of course not.  Thanks for the catch.

Done

http://codereview.appspot.com/1986042/show

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


Re: Extra check in define-event-class patch

2010-08-13 Thread Neil Puttock
On 13 August 2010 09:12, Mike Solomon mike...@ufl.edu wrote:

 Done - thank you for the suggestion.  Attached is the patch series, which
 can also be found on http://codereview.appspot.com/1867050 .  Thank you!

Looks good.  I've added quite a few comments, but just got a sever
error trying to publish them. :(

I'll see if it works tomorrow, otherwise I'll copy them here.

Cheers,
Neil

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


Re: Fix 1198: Fix displayLilyMusic for \time and #(set-time-signature) (issue1986042)

2010-08-13 Thread n . puttock

On 2010/08/13 22:25:43, Carl wrote:

Let me know about the filename for the new callbacks, and I'll add

that to this

patch and push it.


I'd be happy with scm/define-music-callbacks.scm.

Cheers,
Neil

http://codereview.appspot.com/1986042/show

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


Re: woodwind diagrams in lilypond (http://codereview.appspot.com/1385041)

2010-08-13 Thread Mike Solomon
Hey all,

How's this as a simple example?

I will likely be pressed for time over the next week as I start a new job (a
teaching job in which one of the units will be on using Lilypond!). So,
while I won't really be able to take the lead on this feature's
documentation, I can certainly do guided and specific work that would
contribute towards people's understand of these diagrams.  Specifically, if
there's one person who'd like to work on the dox for this, I can be in touch
with him or her directly about how this feature works.

~Mike

On 8/9/10 6:57 PM, Valentin Villenave valen...@villenave.net wrote:

 On Sun, May 30, 2010 at 10:52 PM, Neil Puttock n.putt...@gmail.com wrote:
 On 28 May 2010 07:45, Mike Solomon mike...@ufl.edu wrote:
 2) Do the diagrams look nice?
 
 They're very impressive; well done!
 
 Greetings Mike,
 
 I have just closed the tracker page that had been opened to keep track
 of your woodwind-diagrams patches:
 http://code.google.com/p/lilypond/issues/detail?id=1117
 
 It would be nice if your nice work could be made a bit more visible
 (and helpful) in the Documentation, specifically in
 http://lilypond.org/doc/v2.13//Documentation/notation/common-notation-for-wind
 -instruments
 (at the very least). I'm not sure exactly where it should go
 (fingerings?) nor how to introduce it (a nice, simple example would go
 a long way, btw). Could you have a look?
 
 Cheers,
 Valentin
 



woodwind-diagrams-tutorial.ly
Description: Binary data
___
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel