Re: [PATCH] testsuite: fix parsing of tags used in the testcases

[Ramsay Jones <ramsay@ramsayjones.plus.com>]

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@gmail.com>
> ---
>  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