Hi Andreas,

> On Feb 18 2019, Rainer Orth <r...@cebitec.uni-bielefeld.de> wrote:
>
>> diff --git a/libgo/testsuite/gotest b/libgo/testsuite/gotest
>> --- a/libgo/testsuite/gotest
>> +++ b/libgo/testsuite/gotest
>> @@ -627,13 +627,13 @@ symtogo() {
>>                                   -e 's/[    ]*$/\\n/g' |
>>                               tr -d '\n')"
>>                  # Remove leading and trailing \n.
>> -                output="$(echo "$output" | sed -e 's/^\(\\n\)*//' -e 
>> 's/\(\\n\)*$//')"
>> +                output="$(printf %s "$output" | sed -e 's/^\(\\n\)*//' -e 
>> 's/\(\\n\)*$//')"
>
> printf %s does not print a trailing newline, and some implementations of
> sed cannot cope with incomplete lines.

configure uses

as_echo='printf %s\n'
as_echo_n='printf %s'

here.  To avoid duplicating all that logic to determine a suitable echo,
it would be good to be able to reuse it.

My patch is clearly just a hack to get rid of those failures.

        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to