On 5/14/10 7:01 AM, "Karl Hammar" <k...@aspodata.se> wrote:
> Carl Sorensen:
>> On 5/13/10 1:11 PM, "Karl Hammar" <k...@aspodata.se> wrote:
> ...
>> make test-baseline
> ...
>> make check
> ...
>
> Ok, done that.
>
> With the guidance from http://code.google.com/p/lilypond/issues/detail?id=881:
>
> I can't explain why, but making the print function pure by redefining
> ly:tie::print
> just for LaissezVibrerTie seems to fix this (I haven't done a regtest check,
> so it
> might end up breaking something else):
>
> (define-public (laissez-vibrer::print grob)
> (ly:tie::print grob))
>
> (then add laissez-vibrer::print to pure-print-callbacks)
>
> I tried this:
>
> diff --git a/scm/define-grobs.scm b/scm/define-grobs.scm
> index 176debd..35186f8 100644
> --- a/scm/define-grobs.scm
> +++ b/scm/define-grobs.scm
> @@ -2345,8 +2345,12 @@
> (interval-union '(0 . 0) (cons smaller larger)))
> '(0 . 0))))
>
> +(define-public (laissez-vibrer::print grob)
> + (ly:tie::print grob))
> +
> (define pure-print-callbacks
> (list
> + laissez-vibrer::print
> fret-board::calc-stencil
> note-head::brew-ez-stencil
> print-circled-text-callback
>
> $ make test-redo
>
>> out/test-results/index.html
>>
>> that shows the results of the regression tests.
> ...
>
> I got (except the test-output-distance.ly) an distance of 0300030 and
> "HEAD is: 22d889f4d27469864c31db81445e9de49774ae23" to the right and
> left plus the git-diff to the rigth.
>
> So I assume it was the wrong patch try. Does anybody have Neil's
> proposed fix available ?
You also need to redefine the 'stencil for laissez-vibrez tie in
scm/define-grobs.scm.
HTH,
Carl
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel
