Re: [PATCH 1/2] lsqa.pl: fix --header output

[Zorro Lang <zlang@redhat.com>]

On Tue, Oct 11, 2022 at 11:23:39AM +0200, David Disseldorp wrote:
> 7e98d41a ("fstests: move test group info to test files") replaces the
> post-header "seq" assignment with "./common/preamble\n_begin_fstest ..."
> Update lsqa.pl to recognise the "./common/preamble" line as a header
> divider.
> 
> Signed-off-by: David Disseldorp <ddiss@suse.de>
> ---
>  lsqa.pl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lsqa.pl b/lsqa.pl
> index 8ca1167f..63bee582 100755
> --- a/lsqa.pl
> +++ b/lsqa.pl
> @@ -99,7 +99,7 @@ sub get_qa_header($) {
>  	m/^# SPDX/i		and next; # SPDX tags
>  	m/^# Copyright/i	and next; # Copyright tags
>  	m/^#\s*\-{10}/		and last; # dashed lines
> -	m/^seq/i		and last; # test start
> +	m/^\. \.\/common\/preamble/ and last; # test start

I think that makes sense, I don't have better idea, so ..

Reviewed-by: Zorro Lang <zlang@redhat.com>

>  
>  	s/^# *//;
>  
> -- 
> 2.35.3
>