diff mbox series

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

Message ID 20221011092340.24857-2-ddiss@suse.de (mailing list archive)
State New, archived
Headers show
Series : minor compiler warning and lsqa fixes | expand

Commit Message

David Disseldorp Oct. 11, 2022, 9:23 a.m. UTC 
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(-)

Comments

Zorro Lang Oct. 11, 2022, 3:04 p.m. UTC | #1 
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
>
diff mbox series

Patch

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
 
 	s/^# *//;