----- Original Message ----- From: <[email protected]>
To: <[email protected]>
Cc: <[email protected]>; <[email protected]>
Sent: Wednesday, May 06, 2015 12:40 PM
Subject: Re: Update incipit snippets (issue 235980043 by [email protected])


I haven't checked incipit.ly, but otherwise it looks fine apart from
what appears to be an inadvertent change which needs investigating.


https://codereview.appspot.com/235980043/diff/1/Documentation/snippets/conducting-signs,-measure-grouping-signs.ly
File Documentation/snippets/conducting-signs,-measure-grouping-signs.ly
(right):

https://codereview.appspot.com/235980043/diff/1/Documentation/snippets/conducting-signs,-measure-grouping-signs.ly#newcode31
Documentation/snippets/conducting-signs,-measure-grouping-signs.ly:31:
setting in @@file@{scm/time-signature-settings.scm@}:
Is this change intentional?  The original looks right.  Looks like
someone edited this snippet in Documentation/snippets without adding the
change to snippets/new.

https://codereview.appspot.com/235980043/

Looks like it's a change been made in the LSR, to try to use the file format of TeX, but which gets escaped when it's exported.

I guess the simplest fix is not to bother with @file in the snippet, but just use <code></code>

I'll do this soon, but not necessarily as a part of this specific patch, if that's OK.

--
Phil Holmes

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to