Let parser use define-markup-command-internal

Let parser use define-markup-command-internal

Macros/syntax are quirky to use from C++.


Also:

Split out define-markup-command-internal

[dak, 2018-02-04 16:55:32]

Make do completely without primitive-eval for macro definitions

[thomasmorley651, 2018-02-04 21:36:17]

Works with my guilev2 setup and compiling input/regression/markup-partial.ly
directly.
make doc (with guilev2) fails at different place now, while using -j5.
I'll redo with single thread and report afterwards.

[thomasmorley651, 2018-02-05 00:04:22]

On 2018/02/04 21:36:17, thomasmorley651 wrote:
> Works with my guilev2 setup and compiling input/regression/markup-partial.ly
> directly.
> make doc (with guilev2) fails at different place now, while using -j5.
> I'll redo with single thread and report afterwards.

make doc (guilev2) fails for the code:

\markuplist {
  \override #'(padding . 2)
  \table
    #'(0 1 0 -1)
    {
      \underline { center-aligned right-aligned center-aligned left-aligned }
      one \number 1 thousandth \number 0.001
      eleven \number 11 hundredth \number 0.01
      twenty \number 20 tenth \number 0.1
      thousand \number 1000 one \number 1.0
    }
}

Which is the example of the table-markup-list-command and part of
Documentation/es/changes.tely
Documentation/fr/changes.tely
Documentation/it/changes.tely
scm/define-markup-commands.scm

Error-message:
Interpreting music...
Preprocessing graphical objects...lilypond:
/home/hermann/lilypond-git-guile-2.2/lily/book.cc:258: void
Book::process_score(SCM, Paper_book*, Output_def*): Assertion `0' failed.
Aborted (core dumped)

[dak, 2018-02-05 01:04:22]

On 2018/02/05 00:04:22, thomasmorley651 wrote:
> On 2018/02/04 21:36:17, thomasmorley651 wrote:
> > Works with my guilev2 setup and compiling input/regression/markup-partial.ly
> > directly.
> > make doc (with guilev2) fails at different place now, while using -j5.
> > I'll redo with single thread and report afterwards.
> 
> make doc (guilev2) fails for the code:
> 
> \markuplist {
>   \override #'(padding . 2)
>   \table
>     #'(0 1 0 -1)
>     {
>       \underline { center-aligned right-aligned center-aligned left-aligned }
>       one \number 1 thousandth \number 0.001
>       eleven \number 11 hundredth \number 0.01
>       twenty \number 20 tenth \number 0.1
>       thousand \number 1000 one \number 1.0
>     }
> }
> 
> Which is the example of the table-markup-list-command and part of
> Documentation/es/changes.tely
> Documentation/fr/changes.tely
> Documentation/it/changes.tely
> scm/define-markup-commands.scm
> 
> Error-message:
> Interpreting music...
> Preprocessing graphical objects...lilypond:
> /home/hermann/lilypond-git-guile-2.2/lily/book.cc:258: void
> Book::process_score(SCM, Paper_book*, Output_def*): Assertion `0' failed.
> Aborted (core dumped)

Not a Guilev2 problem.  The same with Guilev1.  Minimal example:

\markuplist {
  \huge
  \column-lines { }
}

So I know what to look for.  Interestingly no regtest triggers.

[dak, 2018-02-05 01:20:14]

Fix result of make-...-markup-list to return a list as the strange API requires

[thomasmorley651, 2018-02-06 21:57:23]

Out of my own testings as well as running a full `make doc' with guilev2:
LGTM