On 11/12/2020 13:06, Luc Van Oostenryck wrote: > In testcases' tags, if a value contains 'check-' then this > value will be used as the tagname instead of the value. > > Fix this by adding a bit more context in the regexp used for parsing these. > > Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> > --- > validation/test-suite | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/validation/test-suite b/validation/test-suite > index 6935d40cee51..1b05c75e9f74 100755 > --- a/validation/test-suite > +++ b/validation/test-suite > @@ -84,8 +84,8 @@ get_tag_value() > check_assert="" > check_cpp_if="" > > - lines=$(grep 'check-[a-z-]*' $1 | \ > - sed -e 's/^.*\(check-[a-z-]*:*\) *\(.*\)$/\1 \2/') > + lines=$(grep '^ \* check-[a-z-]*' $1 | \ > + sed -e 's/^ \* \(check-[a-z-]*:*\) *\(.*\)$/\1 \2/') > > while read tag val; do > #echo "-> tag: '$tag'" > I needed to apply this to get the testsuite to pass on tonights 'fetch'. However, my immediate thought was 'what if you had more than one space before/after the '*'? It just so happens that I had a couple of 'warning: <file>: test unhandled' after applying the patch (for asm-empty-clobber.c and asm-goto-labels.c) where they had two spaces after the '*', so ... :-D ATB, Ramsay Jones
