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