commit 9f96b81ba67f80c2e55581775648b1b9e792cdf9
author Jonas Karlman <jonas@kwiboo.se> Tue Jul 18 20:34:34 2023 +0000
committer Simon Glass <sjg@chromium.org> Wed Aug 02 12:05:57 2023 -0600
tree 74c420ce803661521668a939ca5721c61788aa96
parent 6fa277bbf133baaf4a687a1261a789de5bb188a7

binman: Update missing optional external blob warning text

Make it more clear that the missing external blob is optional in the
printed warning message.

Signed-off-by: Jonas Karlman <jonas@kwiboo.se>
Reviewed-by: Simon Glass <sjg@chromium.org>

tools/binman/control.py

diff --git a/tools/binman/control.py b/tools/binman/control.py
index d1ee1d6..8bc666a 100644
--- a/tools/binman/control.py
+++ b/tools/binman/control.py
@@ -674,7 +674,7 @@
     image.CheckOptional(optional_list)
     if optional_list:
         tout.warning(
-            "Image '%s' is missing external blobs but is still functional: %s" %
+            "Image '%s' is missing optional external blobs but is still functional: %s" %
             (image.name, ' '.join([e.name for e in optional_list])))
         _ShowHelpForMissingBlobs(optional_list)
 

tools/binman/ftest.py

diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py
index 1cfa349..36c72be 100644
--- a/tools/binman/ftest.py
+++ b/tools/binman/ftest.py
@@ -6390,7 +6390,7 @@
         err = stderr.getvalue()
         self.assertRegex(
             err,
-            "Image '.*' is missing external blobs but is still functional: missing")
+            "Image '.*' is missing optional external blobs but is still functional: missing")
 
     def testSectionInner(self):
         """Test an inner section with a size"""