Re: [PATCH 06] parallel-tests: Fix summary output.

[Ralf Wildenhues]

My changes to patch 01 introduced a sed s,...,\n,  command, which is of
course not portable.  It led to bogus summary output on stdout with
Solaris sed and others, as documented by POSIX.

Cheers,
Ralf

    parallel-tests: Fix summary output.
    
    * lib/am/check.am (am__text_box): Fix unportable sed script,
    replacing `\n' in the right hand side of an `s' command with
    a literal newline.  Kudos to Bruno Haible for the newline idea.
    * tests/parallel-tests.test: Update test to expose this.

diff --git a/lib/am/check.am b/lib/am/check.am
index 77e6271..c01a25a 100644
--- a/lib/am/check.am
+++ b/lib/am/check.am
@@ -75,7 +75,8 @@ am__rst_title   = sed 's/.*/   &   
/;h;s/./=/g;p;x;p;g;p;s/.*//'
 am__rst_section = sed 'p;s/./=/g;p;g'
 
 # Put stdin (possibly several lines separated by ".  ") in a box.
-am__text_box = sed 's/\.  /\n/g' | sed '/^$$/d' |              \
+am__text_box = { nlinit=`echo 'nl="'; echo '"'`; eval "$$nlinit"; \
+  sed "s/\\.  /\\$$nl/g"; } | sed '/^$$/d' |                   \
 $(AWK) '{ if (final) final = final "\n" $$0; else final = $$0; }\
 max < length($$0) { max = length($$0); }                       \
 END {                                                          \
diff --git a/tests/parallel-tests.test b/tests/parallel-tests.test
index 6e4b878..2482f30 100755
--- a/tests/parallel-tests.test
+++ b/tests/parallel-tests.test
@@ -70,6 +70,8 @@ test `grep -c '^FAIL' mylog.log` -eq 1
 test -f baz.log
 test -f bar.log
 test -f foo.log
+# The summary should be formatted correctly.
+grep 'failedn' stdout && Exit 1
 
 # clean should remove all log files (but not more).
 : > unrelated.log