[SRU][J][PATCH] UBUNTU: [Packaging] config-check: Handle new annotations format 4

Juerg Haefliger juerg.haefliger at canonical.com
Wed May 31 09:42:02 UTC 2023


For derivatives, we could end up in a situation where the master has
already switched over to the new annotations format 4 but the derivative
still wants to use the old scheme. In that case, the config-check script
is still used and needs to properly handle the new annotations format 4
(in case the derivative includes the annotations from the master).

Ignore: yes
Signed-off-by: Juerg Haefliger <juerg.haefliger at canonical.com>
---
 debian/scripts/config-check | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/debian/scripts/config-check b/debian/scripts/config-check
index df0cd52069b4..6e64277da00b 100755
--- a/debian/scripts/config-check
+++ b/debian/scripts/config-check
@@ -58,12 +58,12 @@ sub read_annotations {
 	die "$P: $filename: open failed -- $! -- aborting\n";
     while (<$fd>) {
 	if (/^# FORMAT: (\S+)/) {
-	    die "$P: $1: unknown annotations format\n" if ($1 != 2 && $1 != 3);
+	    die "$P: $1: unknown annotations format\n" if ($1 < 2 || $1 > 4);
 	    $form = $1;
 	}
 
-	# Format #3 adds the include directive on top of format #2:
-	if ($form == 3 && /^\s*include(\s|$)/) {
+	# Format #3 and #4 add the include directive on top of format #2:
+	if ($form >= 3 && /^\s*include(\s|$)/) {
 	    # Include quoted or unquoted files:
 	    if (/^\s*include\s+"(.*)"\s*$/ || /^\s*include\s+(.*)$/) {
 		# The include is relative to the current file
-- 
2.37.2




More information about the kernel-team mailing list