2017-03-26 10:06 GMT+02:00 David Kastrup <d...@gnu.org>:
> Thomas Morley <thomasmorle...@gmail.com> writes:
>
>> 2017-03-26 0:48 GMT+01:00 James <p...@gnu.org>:
>>> Hello
>>>
>>> After 5099 was pushed I am now seeing problems with basic 'make'.
>
>>
>> accidently I permanently deleted the branch where the patch for issue 5099
>> was.
>> But I had it as git-formated patch, so I reapplied it to a new branch
>> and pushed from there.
>>
>> I've now retested make successfully, though.
>>
>> So I've no clue what's wrong.
>
> Huh?
>
> diff --git a/scm/lily-library.scm b/scm/lily-library.scm
> index c6f066ca32..1b3926af7f 100644
> --- a/scm/lily-library.scm
> +++ b/scm/lily-library.scm
> @@ -778,9 +778,9 @@ as rectangular coordinates @code{(x-length . y-length)}."
>
> (define-public (remove-whitespace strg)
> "Remove characters satisfying @code{char-whitespace?} from string @var{strg}"
> - (string-delete
> - strg
> - char-whitespace?))
> + (if (guile-v2)
> + (string-delete char-whitespace? strg)
> + (string-delete strg char-whitespace?))
>
> (define-public (string-encode-integer i)
> (cond
>
> The deleted lines have a net paren change of -1 (one paren more closed
> than opened). The added lines have a net paren change of 0.
>
> This cannot possibly have worked.
Indeed.
How could it survive automatic tests then?
>
> But that's issue 5101 according to the commit message.
Yep.
5099 looks still ok to me.
> Backing out that patch from staging, and taking a further look.
How to proceed?
Upload a fixed patch for review?
I'm terribly sorry for the additional work for you and James,
Harm
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
