On 28/06/20 23:30, Havard Skinnemoen wrote: > If a test is unsuccessful, the result is "not ok", which does not match > the regex because it includes a space. > > This regex matches both "ok" and "not ok". > > Signed-off-by: Havard Skinnemoen <hskinnem...@google.com> > --- > tests/Makefile.include | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/Makefile.include b/tests/Makefile.include > index 3f4448a20b..09df2d3f86 100644 > --- a/tests/Makefile.include > +++ b/tests/Makefile.include > @@ -637,7 +637,7 @@ define do_test_tap > { export MALLOC_PERTURB_=$${MALLOC_PERTURB_:-$$(( $${RANDOM:-0} % > 255 + 1))} $2; \ > $(foreach COMMAND, $1, \ > $(COMMAND) -m=$(SPEED) -k --tap < /dev/null \ > - | sed "s/^[a-z][a-z]* [0-9]* /&$(notdir $(COMMAND)) /" || true; ) > } \ > + | sed "s/^\(not \)\?ok [0-9]* /&$(notdir $(COMMAND)) /" || true; > ) } \ > | ./scripts/tap-merge.pl | tee "$@" \ > | ./scripts/tap-driver.pl $(if $(V),, --show-failures-only), \ > "TAP","$@") >
Queued, thanks. Paolo