annotations: update: properly update note for arch-flavour configs
authorAndrea Righi <andrea.righi@canonical.com>
Mon, 30 Oct 2023 13:08:49 +0000 (14:08 +0100)
committerAndrea Righi <andrea.righi@canonical.com>
Mon, 11 Dec 2023 10:00:49 +0000 (11:00 +0100)
Values in the form "arch-flavour" are incorrectly processed in the logic
to determine if the note needs to be updated, causing a bunch of notes
to be incorrectly replaced with 'TODO: update not'.

Fix this by searching the values with the proper arch and flavour.

Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
kconfig/annotations.py

index 145523ce03633931dffa8e9fee82cb204aef8c07..5cd51ae2bb6b3168a5d552104041788a087b19d8 100644 (file)
@@ -227,6 +227,7 @@ class Annotation(Config):
                 }
 
         # Import configs from the Kconfig object into Annotations
+        flavour_arg = flavour
         if flavour is not None:
             flavour = arch + f"-{flavour}"
         else:
@@ -240,7 +241,7 @@ class Annotation(Config):
                 if "policy" in self.config[conf]:
                     # Add a TODO if a config with a note is changing and print
                     # a warning
-                    old_val = self.search_config(config=conf, arch=arch, flavour=flavour)
+                    old_val = self.search_config(config=conf, arch=arch, flavour=flavour_arg)
                     if old_val:
                         old_val = old_val[conf]
                     if val != old_val and "note" in self.config[conf]: