Re: Issue 4504/5 update changes.tely (issue 275770043 by address@hidden)

[paulwmorris]

Thanks for your help.  I made a test file and got it to compile without
errors.


https://codereview.appspot.com/275770043/diff/40001/Documentation/changes.tely
File Documentation/changes.tely (right):

https://codereview.appspot.com/275770043/diff/40001/Documentation/changes.tely#newcode88
Documentation/changes.tely:88: g1'
On 2015/11/03 22:21:55, dak wrote:
> This needs to be g'1 instead of g1' .

Yikes. How did I mess that up...

https://codereview.appspot.com/275770043/diff/40001/Documentation/changes.tely#newcode91
Documentation/changes.tely:91:
On 2015/11/03 21:45:03, pkx166h wrote:
> This snippet doesn't compile. I cannot work out why.

> Remember that this is all built using lilypond-book during doc
compilation, so
> you can test any snippet you add by using a basic texinfo file layout
and adding
> the TexInfo code:

Thank you for showing me that; it's very helpful.  I made a test file,
corrected the g1' -> g'1, and fixed another problem.  This:

  \override #'(whiteout-style . outline)

should actually be this:

  \override #'(style . outline)

(Originally I was using 'whiteout-style' for the markup command
property, but changed it to just 'style' because other markup commands
use 'style', but forgot to update it here. (groan...))

The good news is that with these fixes, and running it with the compiled
version, everything went fine.  Log output below.  So I will submit
another patch set with these changes.


[build (whiteout-pad)]$ out/bin/lilypond-book --pdf mytest/test.tely
lilypond-book (GNU LilyPond) 2.19.31
Reading mytest/test.tely...
Running texi2pdf on file /tmp/tmpuL_Y2D.texi to detect default page
settings.

Dissecting...
Writing snippets...
Processing...
Running lilypond...
GNU LilyPond 2.19.31
Processing `./snippet-map-513811304.ly'
Parsing...
Processing `./63/lily-ffa13262.ly'
Parsing...
Interpreting music...
Preprocessing graphical objects...
Calculating line breaks...
Drawing systems...
Layout output to `./63/lily-ffa13262.eps'...
Converting to `./63/lily-ffa13262.pdf'...
Converting to PNG...
Layout output to `./63/lily-ffa13262-1.eps'...
Layout output to `./63/lily-ffa13262-2.eps'...
Layout output to `./63/lily-ffa13262-3.eps'...
Converting to `./63/lily-ffa13262-1.pdf'...
Converting to `./63/lily-ffa13262-2.pdf'...
Converting to `./63/lily-ffa13262-3.pdf'...
Writing ./63/lily-ffa13262-systems.texi...
Writing ./63/lily-ffa13262-systems.tex...
Writing ./63/lily-ffa13262-systems.count...
Success: compilation successfully completed
Linking files...
Compiling /home/paul/lilypond-git/build/test.texi...
Writing `/home/paul/lilypond-git/build/test.texi'...
[build (whiteout-pad)]$

https://codereview.appspot.com/275770043/