bug-lilypond
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: commit 12fe78825798191ecb7e5a4ee3064679773ae1ab is broken


From: David Kastrup
Subject: Re: commit 12fe78825798191ecb7e5a4ee3064679773ae1ab is broken
Date: Sun, 10 Jun 2018 18:10:39 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)

"James Lowe" <address@hidden> writes:

> Knut,
>
> On Sat, 9 Jun 2018 23:19:01 +0200, Knut Petersen
> <address@hidden> wrote:
>
>> Hi James!
>> > Checking out the commit before any of those three that Malte
>> > checed in and then I built LilyPond and simply ran
>> >
>> > lilypond -png $LILYPOND_GIT/input/regression
>> 
>> encodingdefs.ps is unused if neither '-O TeX' nor '-O TeX-GS' is
>> used on the command line.
>> 
>> Checkout, make and install 12fe788,  create a file test.ly:
>> 
>>     \version "2.21.0"
>>     {c''-.}
>> 
>> Then compile with
>> 
>>     lilypond -O TeX --png --pdf test.pdf
>
> Thanks.
>
> I appreciate the time you have taken with me to explain this.
>
> I went back and re-did the 'tests' I would have done with Malte's patch.
>
> And now I can see it - the incorrect pizzicato glyph does show up. I
> simply must have missed this when testing Malte's patch.
>
> All I can do now is apologise to you (for all the extra explaining I
> made you do) and to Malte for not spotting this at the time and the
> rest of the dev team allowing this through. I guess I am just so used
> to using the test-distance reg diff showing up only to show me the
> tests completed 'properly' - i.e if I don't see this reg test diff
> show up I know the patch has gone wrong.

Uh, the procedures are designed as a safety net, not a trampoline.

The actually surprising thing is how much relevant feedback you are able
to provide.

-- 
David Kastrup



reply via email to

[Prev in Thread] Current Thread [Next in Thread]